NGEET / tools-fates-landusedata

Python tool for processing LUH2 data for use with FATES Landuse run modes
Other
0 stars 2 forks source link

`RegridLoop` does not iterate to the last index in the LUH2 dataset variable list #15

Open glemieux opened 1 month ago

glemieux commented 1 month ago

The variable varlen does not need to be reduced by one as range will generate an iterable object starting from 0. https://github.com/NGEET/tools-fates-landusedata/blob/5b357329214ac3b647daa3d471fb9fac9c116a54/src/landusedata/regrid.py#L36-L39

This is a silent and "harmless" bug as the last index in the LUH2 datasets are not variables that are regridded and thus skipped anyways. That said, this should be fixed should the variable order change.