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

revision agricultural demand and production #124

Merged
merged 20 commits into from
Sep 6, 2024

Conversation

flohump
Copy link
Contributor

@flohump flohump commented Sep 6, 2024

No description provided.

Copy link
Member

@danielhuppmann danielhuppmann left a comment

Choose a reason for hiding this comment

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

Thanks for adding the subcategories!

I made a lot of suggestions (which you can accept in bulk) for consistently using capitalized variable names.

Also, in the agmip- attribute, please either use the right agmip name where you added sub-categories or replace the top-level category by ???.

definitions/variable/agriculture/agriculture.yaml Outdated Show resolved Hide resolved
definitions/variable/agriculture/agriculture.yaml Outdated Show resolved Hide resolved
definitions/variable/agriculture/agriculture.yaml Outdated Show resolved Hide resolved
definitions/variable/agriculture/agriculture.yaml Outdated Show resolved Hide resolved
definitions/variable/agriculture/agriculture.yaml Outdated Show resolved Hide resolved
definitions/variable/agriculture/agriculture.yaml Outdated Show resolved Hide resolved
definitions/variable/agriculture/agriculture.yaml Outdated Show resolved Hide resolved
definitions/variable/agriculture/agriculture.yaml Outdated Show resolved Hide resolved
definitions/variable/agriculture/agriculture.yaml Outdated Show resolved Hide resolved
agmip: Production (PROD) - Livestock (LSP)
tier: 3
- Agricultural Production|Livestock|Non-ruminant:
definition: Production of livestock products
Copy link
Member

Choose a reason for hiding this comment

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

Please add an explanation what a non-ruminant is for an energy person like me...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure, I can copy-paste this from the demand definition

flohump and others added 17 commits September 6, 2024 16:59
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Co-authored-by: Daniel Huppmann <dh@dergelbesalon.at>
Copy link
Member

@danielhuppmann danielhuppmann left a comment

Choose a reason for hiding this comment

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

Nice work, thank you!

@danielhuppmann danielhuppmann merged commit 6925b80 into IAMconsortium:main Sep 6, 2024
3 checks passed
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.

2 participants