-
Notifications
You must be signed in to change notification settings - Fork 175
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
Sorg release #7429
Sorg release #7429
Conversation
rewriting ProvdeNRetranslocation function - debugging as it introduced a change
…ies so I used JabRef to detect duplicates. I cleaned up 50 existing ones and approximately 20 from the new list. The order looks to have been altered as well.
@hol353 This can be merged and released. The documentation is in a usable state - I will keep making some improvements to it when I get a chance but it doesn't need to hold it up any longer. |
@jbrider Well done on getting the sorghum model ready for release. I don't want to hold it up but Neil and I have a couple of questions about the sorghum model:
I don't think any of this would take long as there are existing examples that you can copy and modify. |
The SorghumPlausibilityRuns files has 6 simulations in it including:
I will add a location example to that file as well. |
Thanks @jbrider I didn't see the separate SorghumPlausibilityRuns .apsimx file. A lot of work has gone into it which is great! Well done. Add a location example to it and I'll move the files from UnderReview to the validation directory and merge this PR. |
Added extreme environment tests to plausibility file
@hol353 This is right to merge. |
retest this please jenkins |
1 similar comment
retest this please jenkins |
Sorghum is released. Well done @jbrider and the sorghum team! |
working on #572