Julia Signell
Julia Signell
> (But perhaps it should be a config option, in case there are downsides?) I agree that it should be a config option. Especailly since it's a big change in...
@AndreaGiardini I wasn't sure what the best approach would be to make this configurable. Do you have ideas about how this functionality should work?
Yeah this PR is really meant as a starting point and is not mergeable until there is a config setting. I think a context manager sounds like a much better...
Regarding deprecation cycle. I think something like: - add a setting option that turns on the new defaults so that people can opt in prematurely - If user does not...
Well as you might have guessed. This didn't end up happening at the STAC sprint :disappointed: but I am still optimistic that I'll be able to chip away at it...
I'll put up a PR so pystac can be ready as soon as there is a release.
Items don't have this kind of arbitrary get logic though right? So it might be that we add a bunch of new properties to the Asset class.
I'm starting to tackle these with the top priority being the cli
It looks like you are passing an `xarray.DataArray` object (`ds[var]`) as `x` and `y` when you should be passing a string (`var`, or `'air_temperature_2m'`). I am wondering what you are...
I think it is not building from main because the link on "Edit on GitHub" points to master.