Conversation
d-v-b
commented
Dec 8, 2025
- eagerly compute multiscales
- directly copy chunk bytes and metadata documents
…to perf/direct-zarr-io
|
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
…to perf/direct-zarr-io
| s2_parser.add_argument( | ||
| "--omit-nodes", help="The names of groups or arrays to skip.", default="", type=str | ||
| ) |
There was a problem hiding this comment.
this argument solves #81. You would pass --omit-nodes "quality/l2a_quicklook" to omit that group
|
@emmanuelmathot the redundant multiscales calculation is now fixed, and the chunk sizes / sharding are now consistent with the design goal (use as few objects as possible). On my local system, using dask for rechunking was much slower than what I am currently doing (plain assignment via zarr python). |
|
since we are re-encoding the zarr groups here, I can also handle the NaN conversion in this branch, unless that's better in a separate branch @emmanuelmathot |
|
with a1375b7 we have an option (defaulting to false) of allowing invalid values (nan and inf) in the output. When set to |
…to perf/direct-zarr-io
…odel into perf/direct-zarr-io
|
Just tested the last version of this PR: https://api.explorer.eopf.copernicus.eu/raster/collections/sentinel-2-l2a-staging/items/S2B_MSIL2A_20251115T091139_N0511_R050_T35SLU_20251115T111807/viewer |
|
i'll have a look later today! |
|
@d-v-b it seems multiscales are working now but can you tell me wwhat is the fill value? It seems it has been changes (black part that should be transparent) |
…e name and comment out verbose flag
|
seems also that we are missing the fixedscaleoffset codec declaration. I dont understand how xarray resolves correctly the data values. |
…to perf/direct-zarr-io
