OK #639 has too many changes in it to unambiguously evaluate model behavior changes, so:
This PR isolates the issue we know we need to fix — #643 . Basically, the interpolated nature of the LUC emissions and uptake time series means that we can emit or take up more/less than specified by the input data.
NOTE: currently both the old-new test (to easily see errors) and the post-LUC-pulse test (because not taking out C proportionately) are disabled. ~The former needs to be re-enabled before merging.~
bpbond
commented
1 year ago 
OK #639 has too many changes in it to unambiguously evaluate model behavior changes, so:
This PR isolates the issue we know we need to fix — #643 . Basically, the interpolated nature of the LUC emissions and uptake time series means that we can emit or take up more/less than specified by the input data.
NOTE: currently both the old-new test (to easily see errors) and the post-LUC-pulse test (because not taking out C proportionately) are disabled. ~The former needs to be re-enabled before merging.~