Addition of Agroforestry Model

[fainges]

An agroforestry model is required. Implementation will be by Neil Huth and Justin Fainges. The implementation will involve the creation of a multi-Zone aware tree model and associated GUI.

[fainges]

Moved model to Prototypes directory.

[hol353]

This is the initial reviewers report by Michael Battaglia dated 30th August 2016
Agroforestry_System_Model_APSIM_ReviewersReport_v1 3.zip

[sarahcleary]

@hut104 - can you please provide an update on where this is at?

[sarahcleary]

@hol353 to discuss progress with @hut104

[hol353]

I'm happy for Agroforestry to be included in the release.

[jbrider]

Approved.

…

On Thu, Jan 30, 2020 at 9:28 AM Dean Holzworth ***@***.***> wrote: I'm happy for Agroforestry to be included in the release. — You are receiving this because you are on a team that was mentioned. Reply to this email directly, view it on GitHub <#56?email_source=notifications&email_token=AAE3G7BPM72AKBHODDLWQELRAIGK7A5CNFSM4BA6HKXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKJEP4A#issuecomment-580012016>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAE3G7BBMIOF6FFC63PPAQTRAIGK7ANCNFSM4BA6HKXA> .

[sarahcleary]

@APSIMInitiative/reference-panel - still awaiting approval from AgR and USQ

[LouisAK]

Approved

[sno036]

I will try to look at this before the meeting but tomorrow is pretty packed - might be later in the week

[sarahcleary]

@sno036 - just a reminder we are awaiting AgR's approval.

[sarahcleary]

@sno036 - just a reminder we are awaiting AgR's approval.

[sno036]

@hut104 - sorry - I don't think this is ready for release. I'm very interested to have a good look at it but it needs documentation.

I can't see any documentation (which is a problem to begin with),
I opened the Single Tree Example simulation:
• the last parameter on the zone has a spelling error and the unit is obscure (H = multiples of height? I assume. But it needs some explanation)
• Having rainfall interception as a straight fraction of rainfall is a really bad idea – the fraction should be relatively large for small rainfall events but small for large rainfall events – there is a maximum interception possible. Or is this supposed to be stemflow or something? Either way I think it needs some more work (even if it just relabeling as stemflow) (and some documentation).

• I didn't investigate much further because there wasn't any documentation so I didn't know what I was doing.

[sarahcleary]

Any update @hut104?

[hut104]

It's my understanding that this is now released.

…

________________________________ From: sarahcleary <[email protected]> Sent: Tuesday, 31 March 2020 10:26 PM To: APSIMInitiative/ApsimX <[email protected]> Cc: Huth, Neil (A&F, Toowoomba) <[email protected]>; Mention <[email protected]> Subject: Re: [APSIMInitiative/ApsimX] Addition of Agroforestry Model (#56) Any update @hut104<https://github.com/hut104>? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#56 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AC2UVWUJ7MRVRGISNTCTGH3RKHOO3ANCNFSM4BA6HKXA>.

[sarahcleary]

@hol353 - where is this at?

[hol353]

Looks like this has made it into release but it can still be improved upon. The documentation seems to be fixed and is available on the web site.

[sarahcleary]

@sno036 - I assume this can be removed from the RP agenda? please let me know if it needs further discussion

[sarahcleary]

In Release