Branch devel_v8.0.0 currently uses SOILWAT2 units provided by user via file "SW2_netCDF_output_variables.tsv" for output.
For instance, this will make the code to assume that the value of temp_max is in degC (which is correct) and then converts it to degF for output in nc-mode.
SW2 variable
SW2 units
...
netCDF units
temp_max
degC
...
degF
This is risky. Nothing prevents a user from changing degC to degF. Then the code would assume that the value of temp_max is already in units of degF (but that is incorrect).
Solution:
--> hard-code units of SOILWAT2 variables
--> SW_NC_read_out_vars() issues a warning if units obtained from "SW2_netCDF_output_variables.tsv" as input[SWUnitsInd] disagree with hard-coded units units_sw
Branch devel_v8.0.0 currently uses SOILWAT2 units provided by user via file
"SW2_netCDF_output_variables.tsv"for output.For instance, this will make the code to assume that the value of
temp_maxis indegC(which is correct) and then converts it todegFfor output in nc-mode.This is risky. Nothing prevents a user from changing
degCtodegF. Then the code would assume that the value oftemp_maxis already in units ofdegF(but that is incorrect).Solution: --> hard-code units of SOILWAT2 variables -->
SW_NC_read_out_vars()issues a warning if units obtained from"SW2_netCDF_output_variables.tsv"asinput[SWUnitsInd]disagree with hard-coded unitsunits_sw