Florian Humpenöder

We also need to adjust this function in the magpie4 R library. The output is used by REMIND. https://github.com/pik-piam/magpie4/blob/master/R/EmissionsBeforeTechnicalMitigation.R I will coordinate with @bodirsky .

> We also need to adjust this function in the magpie4 R library. The output is used by REMIND. > https://github.com/pik-piam/magpie4/blob/master/R/EmissionsBeforeTechnicalMitigation.R > I will coordinate with @bodirsky . I updated...

> A few documentation changes. Seems good to me, but should be done analogous for all 3 realizations. Please let me know if you need help with understanding the penalty...

I think clarity is essential here. I suggested including both definitions because it helps make transparent that different interpretations of "yield" exist and are in use across modeling teams. Including...

Thanks for the suggestion. Indeed that seems the better solution. I will adjust the PR accordingly.

@danielhuppmann The test stops with `Illegal character(s) '{', '}' in components of variable 'Land Cover|Cropland'.` It seems that {} don't work in components if the brackets are at the end....

For REMIND-MAgPIE we will be able to report fallow land in the future. I understood that fallow land is a larger factor in AIM. Therefore, it would be helpful to...

> Code looks good, some minor questions. Just to clarify, we have 62 age classes now (up from 33); you meant the oldest age class before acx is now ac300?...

> Just from looking at the plots, any idea why there are less CO2 emissions with this new implementation @flohump ? Is that simply due to the wrong unit? Before...

> > > Just from looking at the plots, any idea why there are less CO2 emissions with this new implementation @flohump ? Is that simply due to the wrong...