Skip to content

add xarray-erddap entry-point#434

Merged
ocefpaf merged 1 commit intoioos:mainfrom
ocefpaf:xarray-erddap
Feb 6, 2026
Merged

add xarray-erddap entry-point#434
ocefpaf merged 1 commit intoioos:mainfrom
ocefpaf:xarray-erddap

Conversation

@ocefpaf
Copy link
Member

@ocefpaf ocefpaf commented Feb 4, 2026

I'm still not sure if this should be part of erddapy or a stand-alone plugin. I'll leave this one as a draft for now. When re-visiting we need:

@abkfenris
Copy link
Contributor

So far it looks largely small enough that I don't see a great reason not to have the engine in this repo.

@ocefpaf
Copy link
Member Author

ocefpaf commented Feb 5, 2026

So far it looks largely small enough that I don't see a great reason not to have the engine in this repo.

The reason was.... A matter of taste. Some folks are against erddapy and do not want to install it. I don't know why 🤷🏽‍♂️ . My original intent was to give those folks an option of using this without knowing they are using erddapy. Anyway, I don't think I care about this anymore and this will probably be its home.

PS: it should not get bigger than that BTW. It larger uses erddapy under the hood. This mostly makes the .to_xarray() an xarray dataset opener via an erntry-point.

@ocefpaf ocefpaf marked this pull request as ready for review February 6, 2026 17:45
@ocefpaf
Copy link
Member Author

ocefpaf commented Feb 6, 2026

OK Got all the feedback we needed here. Merging this. Thanks all that responded here or elsewhere!

@ocefpaf ocefpaf merged commit 9944a00 into ioos:main Feb 6, 2026
11 checks passed
@ocefpaf ocefpaf deleted the xarray-erddap branch February 6, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants