Closed cboisvenue closed 4 years ago
Interesting note: the disturbance matrices, which are also proportional, do not have that added one. Scott Morken when we built these did not add that initial identity matrix in the same way as as he did with all the other processes. Disturbance matrices which are loaded from the SQL CBM-CFS3 default data base also had the problem that they were not adding to one exactly (range from ~0.9899 to ~1.003). That later problem was fixed in commit 8dadadc (Nov 30, 2020). The former observation means that either they all need the identity matrices (the >1 values for each pool) or the yearly processes with transitional matrices need to be fixed to not have proportions over 1.
problem addressed: all proportional matrices (Disturbances, BioTurnover, DomTurnover, DomDecay, SlowDecay, SlowMixing) rows (source pools) transfers add up to 1. Additive growth matrices have been checked: all carbon coming in comes from row 1 (input, so the atmosphere), while all other rows the value is equal to 1 so no carbon is lost. OvermatureDecline matrices are to be addressed in another issue (#7 ).
some of the matrices in sim$processes add-up to more than 1 (more than 100% of the carbon at time 0 at time 1 in a non-atmospheric input carbon pool) after a transfer. Here is an example, look at row == 17: