WIP: Support icechunk stores#96
Conversation
Co-authored-by: Henry Rodman <henry.rodman@gmail.com>
|
I noticed that some tests fail with in the This seems to only happen with caching is enabled (and only for the netcdf store)! To show this I have added an additional parametrization to the I did confirm that this error is introduced with the upgrade of either xarray or zarr here. When I just add the changes to Ill move on here for now, but I would love to fix this issue here. @hrodmn do you have any intuition why the caching would not play nice with a new xarray or zarr version? |
|
Ok making some progress here, but Ill need some input from folks. I have refactored the test datasets for zarr v2 and v3. The reading in seems to work (since the tests to get the variables only do pass), but I have some questions about the expected responses. when I run the I am not exactly sure how these come about TBH. Are 'band_metadata', Happy to just rewrite the expected output, but I first wanted to understand exactly what is going on. All I could gather here was that these response fields are expected for e.g. the netcdf output. |
pyproject.toml
Outdated
| "s3fs", | ||
| "xarray", | ||
| "zarr>=2,<3", | ||
| "xarray>2025.07.1", |
There was a problem hiding this comment.
The pickling problem is solved if we bump this up to 2025.10.1!
|
Ok I was able to pass the simple tests (including native icechunk) when running from a local modified version of titiler.xarray (now exists as PR here). Just a note to myself how to link uv to a local file. In pyproject.toml: |
|
Almost have all tests passing, but I am not sure how to interpret these: Is there any way to enable the full error trace to be displayed? |
|
Ah thanks to @hrodmn I was able to look at the response detail and get: currently stuck here. I suspect that this is somehow related to upstream changes ( for example here: #96 (comment)) or the way I have regenerated some of the datasets (less likely since this is happening for every test dataset and I only replaced the zarr/icechunk ones). |
|
@hrodmn and I had the suspicion that this is a dependency issue, and I traced down changes by checking out a fresh branch based on main, doing It turns out the errors are introduced by rio-tiler ( |
|
Moving this over to #101 |
I am working towards icechunk (for now likely just native) support in titler-multidim here.
Depends on #95 and developmentseed/titiler#1235
Tasks