NOAA-OWP / t-route

Tree based hydrologic and hydraulic routing
Other
40 stars 45 forks source link

Upper limit on celerity value in diffusive Fortran kernel #756

Closed kumdonoaa closed 2 months ago

kumdonoaa commented 3 months ago

Too high values of the celerity in the diffusive Fortran kernel often lead to too small (sometimes several orders of magnitude smaller than on second), which virtually freezes the kernel. To prevent it from happening, upper limits on celerity are placed after deriving from the Courant condition with minimum simulation time increment being initially given time step divided by an user-provided numeric value (10 in this case).

Additions

diffusive.f90: Once validated, the user-provided numeric value, for better use, should be directly passed from config yaml or through diffusive_util_v02.py

Removals

-

Changes

-

Testing

1.

Screenshots

Notes

-

Todos

-

Checklist

Testing checklist

Target Environment support

Accessibility

Other