Handle land use change uptake fluxes correctly

bpbond commented 2 years ago

This PR

Adds internal data structures and data-reading code for LUC_UPTAKE, paralleling LUC_EMISSIONS; add references to this in the INI files
Change the LUC variables to fluxpools; the model will now stop with an error if negative values are given to either LUC emissions or uptake
Use this new data time series in Hector computations
Allow access to LUC_UPTAKE via the R package
Update comparison data. Because we were previously ignoring the LUC_UPTAKE columns in the emiss-constraints files, scenarios with 21st/22nd century land use change uptake fluxes diverge from previous outputs (see below)

ini

Closes #523

kdorheim commented 2 years ago

sounds good!

bpbond commented 2 years ago

Not sure what's going on with the lint failure, but I don't think it's me. OK to merge even if it's erroring @kdorheim ?

kdorheim commented 2 years ago

yes sounds good to me especially since this is not directly being merged into main

JGCRI / hector