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

update documentation 32_forestry module #4

Merged
merged 3 commits into from
Sep 27, 2018
Merged

update documentation 32_forestry module #4

merged 3 commits into from
Sep 27, 2018

Conversation

flohump
Copy link
Contributor

@flohump flohump commented Sep 26, 2018

No description provided.

core/sets.gms Outdated
@@ -277,7 +277,7 @@ secdforest_vegc, secdforest_litc, secdforest_soilc, urban_vegc, urban_litc,
secdforest_vegc, secdforest_litc, secdforest_soilc, urban_vegc, urban_litc, urban_soilc,
other_vegc, other_litc, other_soilc /

co2_forestry(emis_source_cell) forestry land CO2 emission sources
co2_forestry(emis_source_cell) sources of forestry land CO2 emissions
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

capitalize first letter

@@ -11,10 +11,10 @@ sets
land32 forestry land pools
/ new, new_ndc, prot, grow, old/

pol32 afforestation policy type
pol32 afforestation policy
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

capitalize first letter

/ none, npi, ndc/

ac_land32(ac,land32) mapping age class - land type
ac_land32(ac,land32) mapping between age class and forestry land type
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

capitalize

*' @description In the affore_vegc_dec16 realisation, the forestry sector is static as
*' described in the static realisation. But on top of the existing forestry land in 1995,
*' forestry land can increase in size due to afforestation. Afforestation can be modelled
*' @description The main feature of the affore_vegc_dec16 realisation is afforestation
Copy link
Member

@FelicitasBeier FelicitasBeier Sep 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spell realization with "z" (b/c we spell it like this in the headline of the documenation document)

@flohump
Copy link
Contributor Author

flohump commented Sep 27, 2018

Please review again!

@tscheypidi tscheypidi merged commit 258302b into magpiemodel:release Sep 27, 2018
bodirsky pushed a commit that referenced this pull request Jan 15, 2019
tscheypidi pushed a commit that referenced this pull request Apr 16, 2019
xwangatpik referenced this pull request in xwangatpik/magpie Feb 11, 2023
incorporate SNUPE values for FSEC-China into the macceff realization
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