You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While the COMPUTE_TREELINE and JULY_TAVG_SUPPLIED global parameter options and the July_Tavg soil parameter are described in the online documentation, it is not sufficiently clear to a new user that the July average air temperature needs to be supplied if their simulation does not contain any dates in July.
The text was updated successfully, but these errors were encountered:
I have updated the online documentation as follows:
expanded the description of these options and parameter so that the conditions under which July average air temperature must be supplied are explicitly mentioned (earlier, the reader had to infer the conditions from the description)
added cross links among the definitions of all three of these terms.
The following web pages were updated:
Documentation/GlobalParam.shtml
Documentation/SoilParam.shtml
Actually - this should be marked as a bug, because one of the effects is that the model state is not restored as expected under certain conditions, resulting in non-sensical results, without a clear warning.
Expected behavior:
The model should abort with a descriptive error message if COMPUTE_TREELINE is turned on and JULY_TAVG_SUPPLIED is not supplied (or cannot be calculated, but I suggest that JULY_TAVG_SUPPLIED needs to be true to simplify things).
I think the behavior as implemented by Ted is sufficient for 4.1.2.j, but we should have the model fail for 4.2.
I have issued a pull requests that requires JULY_TAVG_SUPPLIED=TRUE when COMPUTE_TREELINE=TRUE. If this condition is not met, a descriptive abort message is given.
While the COMPUTE_TREELINE and JULY_TAVG_SUPPLIED global parameter options and the July_Tavg soil parameter are described in the online documentation, it is not sufficiently clear to a new user that the July average air temperature needs to be supplied if their simulation does not contain any dates in July.
The text was updated successfully, but these errors were encountered: