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

calc_management insufficiently checks input values for B_LU_BRP, B_SOILTYPE_AGR and B_GWL_CLASS #175

Open
BrentWHR opened this issue Dec 13, 2022 · 0 comments

Comments

@BrentWHR
Copy link
Contributor

The inputs for this function are checked elsewhere when calc_management() is called by the wrapper function so it should not be a problem when OBIC is used normally.
Checkmates should be added to check B_LU_BRP values occur in crops.obic, GWL_CLASS values are from a restricted set of options (all prefixed by "Gt", and B_SOILTYPE_AGR must exist in OBIC::soils.obic$soiltype.

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

No branches or pull requests

1 participant