hector icon indicating copy to clipboard operation
hector copied to clipboard

Cannot retrieve atmosphere -> land C flux for first year (1745)

Open ashiklom opened this issue 6 years ago • 2 comments

The way it is currently implemented (#341), atmosphere -> land C flux is calculated in simpleNbox::stashCValues, which is never called for the very first time step (by default, 1745). As a result, the first year that atmosphere-land C flux can be retrieved is 1746.

ashiklom avatar Jan 27 '20 18:01 ashiklom

@bpbond is this on your radar? Do you know if it could pose a problem for the carbon tacking work?

kdorheim avatar Jul 12 '21 15:07 kdorheim

Not at all on my radar. Thanks. No, I don't think so?

bpbond avatar Jul 12 '21 16:07 bpbond

This is an artifact of the way Hector treats dates: "1746" in the output stream means "1746.0" = "January 1, 1746". So the NBP (in the v3 nomenclature) that we retrieve at this timepoint is for the calendar year 1745 (1/1/1745-12/31/1746).

So, closing, but let's make sure the documentation addresses this point.

bpbond avatar Nov 03 '22 11:11 bpbond