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

For the AW_MEAN regridding method, report the width as 1 instead of NA. #1186

Closed
JohnHalleyGotway opened this issue Aug 23, 2019 · 1 comment
Assignees
Labels
MET: Library Code priority: low Low Priority type: enhancement Improve something that it is currently doing
Milestone

Comments

@JohnHalleyGotway
Copy link
Collaborator

This very minor change in convention make the logic easier for printing useful warning messages to the user. This is similar to the nearest, upper-left, upper-right, lower-left, and lower-right types, where by convention we report 1 instead of NA.

Some DTC staff were confused about whether the regridding width applies to the AW_MEAN option and printing an informative warning message and updating the data/config/README file will help clarify.

@JohnHalleyGotway JohnHalleyGotway added type: enhancement Improve something that it is currently doing component: library code priority: low Low Priority labels Aug 23, 2019
@JohnHalleyGotway JohnHalleyGotway added this to the MET 9.0 milestone Aug 23, 2019
@JohnHalleyGotway JohnHalleyGotway self-assigned this Aug 23, 2019
@JohnHalleyGotway
Copy link
Collaborator Author

Did changes in my local feature_1186_aw_mean branch, tested, merged into develop, and pushed up to github.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MET: Library Code priority: low Low Priority type: enhancement Improve something that it is currently doing
Projects
None yet
Development

No branches or pull requests

1 participant