search

IAMconsortium / concordia

Apache License 2.0
0 stars 3 forks source link

_FillValue needs to have the same datatype as the data #47

Closed coroa closed 6 months ago

coroa commented 6 months ago

Hi @TimotheeBrgs,

we updated the datatype to float32 in the last round of updates (Refer to https://github.com/IAMconsortium/concordia/issues/32#issuecomment-1952600975 and https://github.com/IAMconsortium/concordia/blob/main/CHANGELOG.md#gridding),

and forgot to also adjust the datatype of the _FillValue encoding setting.

Sorry for that, will fix for the next review round which is just around the corner (and being delayed due to outstanding decisions)