Jarmo Kikstra

As an additional comment, I just noticed that under "sdg.yaml" we also already have a few per-capita variables: - Useful Energy|Transportation|Passenger [per capita]: description: Useful energy per capita for passenger...

Thanks for the [comment](https://github.com/IAMconsortium/common-definitions/issues/270#issuecomment-2656280388) with your considerations, @khaeru Indeed, I agree that the Final Energy tree is problematic; it should can be more clear in the variable naming (as you...

Quick one on this: > For some proposed variables I fail to understand their meaning or use case. What is the use case of `Useful Energy|Carbon Management` on top of...

That's another way to solve things for common-definitions; like in the Final Energy tree. But currently we don't have that in https://github.com/IAMconsortium/common-definitions/blob/main/definitions/variable/emissions/emissions.yaml

Oliver agreed this is good to go ahead.

I just see now that in the meantime, other versions have been added too - which are missing these updates; #283 #282. Should probably align before merging. @danielhuppmann you added...

Not sure what exactly is causing the 'illegal character' error: ``` =========================== short test summary info ============================ FAILED tests/test_legacy.py::test_legacy_variables - ValueError: Collected 2 errors: 1. Unexpected character in variable: 'Production|Iron...

OK thanks @vruijven for your thoughts. Now updated. Would be great if we can move this in such that we can use it for JUSTMIP submissions.

@phackstock @danielhuppmann can we go ahead here, or is there any additional changes needed? Note: we could also delete Useful Energy [per capita] variables from the sdg YAML

Two thoughts: 1. (major) I'm not sure I'm convinced about placing all GFED-based burning outside of AFOLU, mainly because to me AWB clearly seems related to Agriculture 2. (minor) I...