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.

Sign up for GitHub

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

Jump to bottom

Precipitation is not reported in mm #13

Closed
adamhsparks opened this issue Jul 8, 2016 · 2 comments
Closed

Precipitation is not reported in mm #13

adamhsparks opened this issue Jul 8, 2016 · 2 comments

Comments

@adamhsparks
Copy link
Member

adamhsparks commented Jul 8, 2016 •
edited
Loading

Precipitation is calculated incorrectly as cm not mm.

This commit fixes that.
8d09f2257f982de4fa6ea492248124acbe0a4fc7

Thanks to Gwenael Giboire (@GwenGitHub) for reporting this bug.
@adamhsparks adamhsparks reopened this Jul 12, 2016
@adamhsparks
Copy link
Member Author

Invalid values were returned.

Missing values in PRCP were not being correctly identified upon import and conversion was off by factor 100.

Bugs fixed with this commit, 1721752e313972c85f2577c7a778eca46052fadb

@GwenGitHub
Copy link

Thanks Adam for your prompt response and updatings on the fly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@adamhsparks @GwenGitHub