Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

integrate titiler.xarray into titiler-xarray #24

Closed
abarciauskas-bgse opened this issue Nov 18, 2024 · 8 comments
Closed

integrate titiler.xarray into titiler-xarray #24

abarciauskas-bgse opened this issue Nov 18, 2024 · 8 comments
Assignees

Comments

@abarciauskas-bgse
Copy link
Contributor

propose a naming convention

@hrodmn
Copy link

hrodmn commented Nov 21, 2024

Analagous PR open in titiler-cmr: developmentseed/titiler-cmr#40
titiler-xarray is up next!

@hrodmn
Copy link

hrodmn commented Nov 27, 2024

PR open in titiler-xarray titiler-multidim! developmentseed/titiler-multidim#72

@j08lue
Copy link

j08lue commented Nov 27, 2024

What a great PR description, @hrodmn, complete with consequences and all 😍

Love the (suggested?) name change to multidim. That is how I am referring to this functionality, too - dynamic tiling of multi-dimensional datasets.

@abarciauskas-bgse
Copy link
Contributor Author

@hrodmn is developmentseed/titiler-multidim#72 ready for merge? We can create another ticket for testing https://dev-titiler-xarray.delta-backend.com once it's merged. And we will need to deprecate https://github.com/NASA-IMPACT/veda-config/blob/main/datasets/cmip6-tas.data.mdx before moving to production.

@hrodmn
Copy link

hrodmn commented Jan 7, 2025

@hrodmn is developmentseed/titiler-xarray#72 ready for merge?

@abarciauskas-bgse We can merge it now if we have decided to stop supporting virtual references in that API for now. It was unclear to me if we wanted to add that feature back in or not - I think it is possible but will take a little bit more work.

@abarciauskas-bgse
Copy link
Contributor Author

Thanks @hrodmn I think since this titiler instance is for VEDA, we should wait to add back kerchunk support until we have an explicit use case for it. The CMIP6 dataset was just a demonstration. @j08lue please overrule me if you disagree.

@j08lue
Copy link

j08lue commented Jan 8, 2025

We can merge it now if we have decided to stop supporting virtual references in that API for now

Yes, let us go ahead with merging. The known users have been informed and we explained a way forward to them. 👌

@hrodmn
Copy link

hrodmn commented Jan 13, 2025

The changes to integrate the new titiler.xarray package into the titiler-xarray deployment are merged and deployed to the dev deployment!

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

No branches or pull requests

3 participants