You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In order to ensure compliance with the JSON spec and readability by zarr-js, we clear existing encodings in the datatree and set the _FillValue for each variable using the set_zarr_encoding utility function. While this is generally good, I am concerned that this could lead to the misinterpretation of data if the dataset already had a different no data value set via _FillValue and mask_and_scale was not used when loading the dataset. It seems worthwhile to add a test for that case.
The text was updated successfully, but these errors were encountered:
maxrjones
changed the title
Add additional test for existing _FillValue in encoding
Add test for existing _FillValue in encoding
Mar 19, 2024
In order to ensure compliance with the JSON spec and readability by zarr-js, we clear existing encodings in the datatree and set the
_FillValue
for each variable using theset_zarr_encoding
utility function. While this is generally good, I am concerned that this could lead to the misinterpretation of data if the dataset already had a different no data value set via_FillValue
andmask_and_scale
was not used when loading the dataset. It seems worthwhile to add a test for that case.The text was updated successfully, but these errors were encountered: