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.

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

Fix mass balance error if fertiliser entered after test #64

Merged
merged 2 commits into from
Jun 23, 2024
Merged

Fix mass balance error if fertiliser entered after test #64

merged 2 commits into from
Jun 23, 2024

Commits on Jun 23, 2024

  1. Do tests and fertiliser events in sequence 

    Dont read in fertiliser dates that are outside of simulation date range
    @HamishBrownPFR
    HamishBrownPFR committed Jun 23, 2024
    Configuration menu
    Copy the full SHA
    7d2c249 View commit details
    Browse the repository at this point in the history

  2. Update tests. Note, the tests are not giving correct answers now beca… 

    …use the change that was required to fix the but with Fert applications means the crops now do their uptake prior to fertiliser application so for the test system they become N stressed in lots of cases. The test results can no longer be relied on
    @HamishBrownPFR
    HamishBrownPFR committed Jun 23, 2024
    Configuration menu
    Copy the full SHA
    6186ea2 View commit details
    Browse the repository at this point in the history