Litter model theory and implementation docs#593
Conversation
…d carcasses to it
|
@jacobcook1995 Partial edits. I've tried to do a couple of things:
If those pushes seem ok, let me know and I'll tackle the last big file on - whatever it was... |
|
@davidorme think it's a real improvement thank you! Only thing is that the |
vgro
left a comment
There was a problem hiding this comment.
I really like how you describe the model and the references you provide for context. Your writing is very clear and easy to follow in most places.
I think the two models could be linked a bit better, it's not quite clear to me what happens where any why; for example why is there no environmental impact on the soil processes describes? Would it be worth starting with the litter model?
Here a few more specific comments:
Theory of biomass decay:
- I am not sure about 'biomass decay' as the main heading, in other places you refer to 'soil' or 'soil microbes' I think this might be a bit confusing
- I think it would be worth dropping above and belowground somewhere
- in the third paragraph, the biggest uncertainty IS / the biggest uncertainties ARE . Also maybe better to say 'in the litter model', just to make clear which one you are talking about. And a small typo in the last sentence: the model PROVIDES
- maybe worth mentioning the significantly less data for below ground in the last paragraph
Theory of the soil:
- maybe add to first sentence a short overview of the content and link to headings
- first paragraph: there seems to be a word missing: In response to this there has been a movement towards using soil carbon pool definitions that defined by measurable physical and chemical properties
- could you say something about the relative contribution of the different carbon pools and the turnover rates? just roughly, is it daily or millennia?
Phosphorous pools
- you make reference to rain forests in the first sentence, can you say something about other systems, too?
the section on microbial representation could be a bit more specific, i.e. mention functional groups or enzymes (or why you decided not to use it)
I think it would be useful to mention somewhere how these nutrients are moved from one pool to the next (going back to residence times and rates) and to other part of the model
Theory of litter decay
- above ground litter: all losses ARE assumed to be from...
I saw that and am sad about it... I wanted this: 🔭 |
| slow decaying litter pools. This indirectly captures the impact of nitrogen and | ||
| phosphorus chemistry on litter decay. | ||
|
|
||
| The rest of this page gives provides details on the specific litter pools, the |
There was a problem hiding this comment.
I think you missed adjusting this to the new layout
vgro
left a comment
There was a problem hiding this comment.
I like the changes that you made and I think it is in a good state as an intermediate overview. I think it's clear that this is not the final version from the intro.
|
I think it's useful to have a common "in progress" admonition box for these pages? I mean, there's a chance they never go away, but then we can just have that at the top and keep the text cleaner. So: |
|
@davidorme I've implemented your suggestions about admonitions, let me know what you think |
David said he was happy with the changes, but I think forgot to approve
Description
This PR adds documentation for the litter model documentation of theory. Let me know a) if what I've added makes sense and b) if there's anything else that would be good to add.
The updated documentation can be viewed here https://virtual-ecosystem.readthedocs.io/en/513-litter-model-documentation/
@TaranRallings as part of this I added some stuff to the animal model docs, let me know if you are happy with that. I'm also completely have for you to rework it as you see fit when you write the animal model documentation.
Fixes #513
Type of change
Key checklist
pre-commitchecks:$ pre-commit run -a$ poetry run pytestFurther checks