Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rainfall dataset resolution is too coarse #220

Open
dfguerrerom opened this issue Oct 12, 2023 · 0 comments
Open

rainfall dataset resolution is too coarse #220

dfguerrerom opened this issue Oct 12, 2023 · 0 comments

Comments

@dfguerrerom
Copy link
Collaborator

dfguerrerom commented Oct 12, 2023

The rainfall dataset is too coarse and presents errors when small areas are processed, triggering a non-sense error to the end user:

File ~/1_modules/se.plan/component/scripts/gee.py:71, in <genexpr>(.0)
     67 print("values", values)
     68 # depending on the scale, values from the histogram can be floats, we'll
     69 # convert them to integers... sometimes we'll show more values than the
     70 # asset really have
---> 71 return sorted(set(int(float(val)) for val in values))

TypeError: float() argument must be a string or a real number, not 'NoneType'

we should display a more meaningful error and definitely find another precipitation data source.

@andreamcmahon

@dfguerrerom dfguerrerom changed the title rainfall dataset is too coarse rainfall dataset resolution is too coarse Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant