Replies: 2 comments
-
Thank you for catching this. A PR with a documentation update is welcome. |
Beta Was this translation helpful? Give feedback.
0 replies
-
First wanted to check my understanding was correct. I'll try to find time. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
The default behaviour of
make_geocube
with categorical data is to append anodata
string to the<varname>_categories
variable. This makes sense from a Python indexing perspective inasmuch as the corresponding integer code for this is -1. This seems to override thefill
argument. Again this makes sense with the use of -1 and the position ofnodata
. But if I'm correct in this, the documentation could be tightened up a little to clarify thatfill
only applies to numerical data.Beta Was this translation helpful? Give feedback.
All reactions