Change all history fields to use the new landunit_mask argument, and remove initial settings of history fields to spval #1347
Labels
blocked: dependency
Wait to work on this until dependency is resolved
blocker
another issue/PR depends on this one
enhancement
new capability or improved behavior of existing capability
Once we are satisfied with how we want to move ahead with history field averaging, we should change all history fields to be consistent with our plan.
Tentatively, this will involve changing all history fields to use the new landunit_mask argument and removing the initial setting of each history field to spval. This will also involve removing any uses of set_*=spval, instead using the appropriate setting for landunit_mask to serve that purpose.
In the meeting a couple of weeks ago a suggestion was raised of just starting with default-on biogeophysical variables. However, thinking about this more, I see a couple of issues with this plan:
We should test to make sure answers don't change from doing this. Doing this with confidence depends on #29 (blocked by #29).
Once we have done the above, we can – and should:
The text was updated successfully, but these errors were encountered: