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

Add cloud and snow probabilities #129

Merged
merged 5 commits into from
Nov 13, 2023
Merged

Add cloud and snow probabilities #129

merged 5 commits into from
Nov 13, 2023

Conversation

matthewhanson
Copy link
Member

Related issues

Description

Adds two new assets for cloud and snow probabilities

Checklist

  • Includes tests
  • Includes documentation
  • Updates CHANGELOG

@philvarner
Copy link
Collaborator

This looks like it changes the default scale instead of the default tolerance?

Copy link
Collaborator

@philvarner philvarner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

discrepancy between the PR description indicating a default tolerance change, and the default scale being changed.

@matthewhanson
Copy link
Member Author

Ah, good catch @philvarner , fixed and pushed.

This was referenced Nov 10, 2023
@matthewhanson matthewhanson merged commit 274b339 into main Nov 13, 2023
5 checks passed
@matthewhanson matthewhanson deleted the mah/128-cloud-snow branch November 13, 2023 14:28
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

Successfully merging this pull request may close these issues.

Add cloud and snow probabilities
3 participants