Skip to content
This repository was archived by the owner on Jun 4, 2025. It is now read-only.

Deprecation notice#11

Merged
m-mohr merged 2 commits into
mainfrom
TomAugspurger-patch-1
Jun 4, 2025
Merged

Deprecation notice#11
m-mohr merged 2 commits into
mainfrom
TomAugspurger-patch-1

Conversation

@TomAugspurger
Copy link
Copy Markdown
Collaborator

This deprecates the xarray-assets extension.

The deprecation notice includes recommendations to use some other extensions and libraries that would need to exist for us to actually have an alternative (I'm not planning to work on those, but chime in here if you want to).

Closes #9

This deprecates the xarray-assets extension.

The deprecation notice includes recommendations to use some other extensions and libraries that would need to exist for us to actually have an alternative (I'm not planning to work on those, but chime in here if you want to).

Closes #9
Copy link
Copy Markdown
Contributor

@m-mohr m-mohr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I didn't expect this to happen but I'm all for it.
Do I understand correctly that the zarr extension is still to be defined? If yes, it's potentially be a topic for the next planned STAC/ZARR sprint.

@TomAugspurger
Copy link
Copy Markdown
Collaborator Author

Do I understand correctly that the zarr extension is still to be defined?

Correct, along with updating xpystac to use it. At that point I think we'd have a clear path forward from everything offered here.

@jsignell
Copy link
Copy Markdown

jsignell commented Jun 2, 2025

Is the intention to wait to merge this until the zarr extension exists? I am seeing projects like https://github.com/EOPF-Sample-Service/xarray-eopf that are relying on this extension, so it would probably be beneficial to publicize this deprecation earlier rather than later.

@m-mohr
Copy link
Copy Markdown
Contributor

m-mohr commented Jun 2, 2025

From my side, we can merge already. Leaving the final decision to @TomAugspurger

@TomAugspurger
Copy link
Copy Markdown
Collaborator Author

I dislike implementing deprecations without a replacement ready to go, but that's primarily in programming languages where it's annoying to get a warning that you can't do anything about, other than silence.

I'm not sure what deprecating a STAC extension does. If it's purely informational, then I think it'd be fine to merge this as long as the deprecation notice includes a request for volunteers to implement the missing pieces.

Comment thread README.md Outdated
@m-mohr
Copy link
Copy Markdown
Contributor

m-mohr commented Jun 4, 2025

I made it a little more clear that the extension still has to be created and that STAC is open for proposals. I think with that I'll merge.

@m-mohr m-mohr merged commit 1e68a58 into main Jun 4, 2025
0 of 2 checks passed
@m-mohr m-mohr deleted the TomAugspurger-patch-1 branch June 4, 2025 08:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

This extension doesn't scale well

3 participants