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

Maize leaf #7620

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Maize leaf #7620

wants to merge 17 commits into from

Conversation

hut104
Copy link
Contributor

@hut104 hut104 commented Nov 14, 2022

working on #6726

@hol353
Copy link
Contributor

hol353 commented Nov 14, 2022

retest this please jenkins

@jbrider
Copy link
Contributor

jbrider commented Nov 14, 2022

@APSIMInitiative/reference-panel I bought this to Graeme's attention and he would like to have a discussion about the maize model given we have an established working Nitrogen model around leaf dynamics.

@hut104
Copy link
Contributor Author

hut104 commented Nov 14, 2022

@jbrider Thanks for that.
Currently I am just converting maize to use simple leaf rather than complex leaf as part of our migration to a more maintainable and usable code base. I am not looking to make any major change to the science at this point as I'm just in a 3-day sprint for code maintenance and was looking to sort out some issues raised by users about stress outputs etc.
However, it is worth checking. Are you able to send a link to the code you are suggesting? Would it be a simple change (e.g. adding a different class)?

@jbrider
Copy link
Contributor

jbrider commented Nov 14, 2022

https://github.com/APSIMInitiative/ApsimX/tree/master/Prototypes/C4Maize
This is some of the main work Drew did for us.
It uses the same leaf structure to Sorghum with some changes in parameterisation - so I doubt it is an easy change for the current Maize model.

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

Successfully merging this pull request may close these issues.

None yet

3 participants