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
Currently the logic accounting for a) iteration between surface and canopy temperature solutions, b) snow step vs model step, and c) distributed precip results in a confusing mess of data structures and loops. This makes adding features and preventing bugs more difficult than otherwise.
The text was updated successfully, but these errors were encountered:
@tbohn - now that we've removed the DIST_PREC code and gone ahead with the other cleanup tasks (#162), do you think there is anything else to do here? If this will be part of the 5.0 release, we need to come up with a path forward for how to clean up surface_fluxes.c.
@tbohnhttps://github.com/tbohn - now that we've removed the DIST_PREC
code and gone ahead with the other cleanup tasks (#162 #162), do you think there is
anything else to do here? If this will be part of the 5.0 release, we need
to come up with a path forward for how to clean up surface_fluxes.c.
—
Reply to this email directly or view it on GitHub #31 (comment).
@jhamman : I am going to move this to "someday". Any change in the iteration sequence is likely to change the answer, something that we don't want as part of 5.0. I'll leave it to @tbohn to take a look and see what change he thinks would be worthwhile for a later milestone
Currently the logic accounting for a) iteration between surface and canopy temperature solutions, b) snow step vs model step, and c) distributed precip results in a confusing mess of data structures and loops. This makes adding features and preventing bugs more difficult than otherwise.
The text was updated successfully, but these errors were encountered: