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

bug fixes #20

Merged

peterlafollette merged 1 commit into master from ajk/soil_param_file_check

Apr 1, 2024

Collaborator

ajkhattak commented Feb 26, 2024 •

edited

Loading

The PR resolves #19 and fixes two bugs related to missing non-optional parameters in the config file.

Additions

None

Removals

None

Changes

bug fixes
more descriptive screen outputs at the initialization step
typo (van Genuchten) corrected

Testing

All existing tests passed

Checklist

PR has an informative and human-readable title
Changes are limited to a single goal (no scope creep)
Code can be automatically merged (no conflicts)
Code follows project standards (link if applicable)
Passes all existing automated tests
Reviewers requested with the Reviewers tool ➡️


          bug fixes, and other minor changes.

e5af979

ajkhattak requested a review from peterlafollette

February 26, 2024 21:01

peterlafollette approved these changes

View reviewed changes

Contributor

peterlafollette left a comment

Looks good to me

peterlafollette merged commit 7d0d559 into master

4 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet