-
Notifications
You must be signed in to change notification settings - Fork 0
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
PRISM ADRIO and Devlogs #159
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See preliminary comments; but I'd also like to get this branch rebased on main.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor change noted inline; but also I discovered a significant issue with the way this is handling zip files. I'll add another comment...
Oh and is there a reason we are rounding values to three decimal places? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Getting there! See inline.
Good to squash merge. |
* Branch fixes and dependency fix. * Handling date errors and messaging. * Zip file corrections. * Block indent fix. * Generator refactor. * Fix small issue with dates and file names. * Validating scope and erroring excluded locations. * Slight bug fix and update demo values
PRISM ADRIO adaptation, allowing a user to retrieve climate attributes at a daily granularity, specifically at a centroid of a given geoid. Two devlogs are included, one being a demo showing how to use the PRISM ADRIO template and one displaying the experimentation of downloading and retrieving values from the PRISM raster data files.