mm/time unit bug

[nicolejkeeney]

Description of PR

Fix bug in code that was causing an error if you tried to retrieve variables with units of mm/time (Evaporation and Maximum Precipitation)

How To Test

Try retrieving the max precip and evaporation variables in different units. Confirm that you can retrieve the data without error and that the unit conversions look okay :D

Type of change

• [x] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [ ] This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

• [ ] Test A
• [ ] Test B

Checklist:

• [ ] My code follows the style guidelines of this project
• [ ] I have performed a self-review of my code
• [ ] I have commented my code, particularly in hard-to-understand areas
• [ ] I have made corresponding changes to the documentation
• [ ] My changes generate no new warnings
• [ ] I have added tests that prove my fix is effective or that my feature works
• [ ] New and existing unit tests pass locally with my changes
• [ ] Any dependent changes have been merged and published in downstream modules

[nicolejkeeney]

Fix unit bug in Maximum Precipitation

[nicolejkeeney]

Fix unit bug in Maximum Precipitation

[nicolejkeeney]

No problems returning data for either max precip or evaporation!

However noticing that the "default" unit for max precip is "mm/hr" and for evaporation it's "mm/d". I'm guessing the evaporation data native unit is designed as such?

Yep these are just the native units of the data!