-
Notifications
You must be signed in to change notification settings - Fork 12
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
Pressure masking netcdf modifying tool #43
Conversation
This is a neat self-contained tool that is useful to for FRE workflows, but it needs more documentation, tests, and it's not working perfectly yet in comparison to the Bronx refinediag script. |
@ceblanton I'm going to go in and change the argparse to click while I review this |
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.
Click is the way arguments are handled, not with argparse!
I'm also not overjoyed about the qty of changes required when adding sub-commands, I have a couple thoughts on how to make that better.... but not for this PR. |
This really needs a single pytest. |
No description provided.