-
Notifications
You must be signed in to change notification settings - Fork 0
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
Comments
Analagous PR open in titiler-cmr: developmentseed/titiler-cmr#40 |
PR open in |
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. |
@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. |
@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. |
Yes, let us go ahead with merging. The known users have been informed and we explained a way forward to them. 👌 |
The changes to integrate the new |
propose a naming convention
The text was updated successfully, but these errors were encountered: