CABLE-LSM / CABLE

Home to the CABLE land surface model and its documentation
https://cable.readthedocs.io/en/latest/
Other
8 stars 4 forks source link

Merge from AM3 - params #402

Open JhanSrbinovsky opened 3 days ago

JhanSrbinovsky commented 3 days ago

This issue covers the merging of code in params/ and utils/ directory from AM3 development.

Null testing against offline will be required here @ccarouge, how do you want to proceed? I'm pretty sure I did this a month or so ago before moving on to the next part of the merge BUT:

  1. I no longer have the output handy
  2. I still haven't used benchcab, I am still using manual pre-cursor to benchcab variety of testing.

Note: Not that it actually changes anything, but technically it does intersect with all applications of CABLE

The overarching issue concerning AM3 merging is here #403

ccarouge commented 2 days ago

What is "null testing" ? Is that the same as regression testing?

JhanSrbinovsky commented 2 days ago

I just mean that the regression testing will/should show there is no difference, as expected