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

Export import varaibles / based on NAVIGATE / ENGAGE #286

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

FlorianLeblancDr
Copy link
Contributor

This PR adds Export Import variables from NAVIGATE / ENGAGE

Some macro sectors are missing but will dynamically change by updating the tag (#279 )
Export|Energy
Export|Industry|Chemicals
Export|Industry|Manufacturing
Export|Industry|Energy
Export|Other
Export|Industry|Energy Intensive

Discarded (can be computed within the database):
Exports|% change per GDP
Export|Developing Country Share
Exports|% change from Ref

@FlorianLeblancDr FlorianLeblancDr requested a review from a team February 19, 2025 13:58
@FlorianLeblancDr
Copy link
Contributor Author

Having second thoughts, I am moving export and import variables to the trade.yaml files
and improve clarity of the use of variable name with a header

@FlorianLeblancDr FlorianLeblancDr marked this pull request as draft February 19, 2025 15:22
@FlorianLeblancDr
Copy link
Contributor Author

Looking at NAVIGATE and ENGAGE "Trade" variables, I further discarded:
"Trade|Emissions|Value|Carbon*" variables (redundant with "Trade|Emissions Allowances*"
"Trade|Gross Import|Secondary Energy|Electricity|*": we can add Export and Import of energy in a further release

@FlorianLeblancDr FlorianLeblancDr marked this pull request as ready for review February 19, 2025 15:54
@FlorianLeblancDr
Copy link
Contributor Author

@volker-krey @macflo8
This a good example on implementing both ISIC sector nomenclature and biophysical nomenclature.
The same can be done for Production, for exampel.

@FlorianLeblancDr FlorianLeblancDr removed the request for review from macflo8 March 3, 2025 14:00
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