Skip to content

Fix cyclic Basin forcing - #2313

Merged
visr merged 5 commits into
mainfrom
transient_basin_forcing
Jun 3, 2025
Merged

visr merged 5 commits into
mainfrom
transient_basin_forcing

Conversation

@SouthEndMusic

@SouthEndMusic SouthEndMusic commented Jun 2, 2025 •

Copy link
Copy Markdown
Collaborator

Fixes #2298, fixes #2291

Notes:

  • Refactored getting tstops from timeseries interpolations in type stable manner
  • Refactored callbacks so that now always all of them are created
  • Running 1000 year cyclic_time model now takes 15 seconds
  • Concentrations cannot yet be periodic (Support cyclic Basin / concentration #2105, is this documented?)
  • Fixed some unit formatting and added reduction factor for resistance nodes in docs

@SouthEndMusic
SouthEndMusic requested a review from evetion June 2, 2025 06:17
@SouthEndMusic SouthEndMusic changed the title Transient basin forcing Fix cyclic Basin forcing Jun 2, 2025
Comment thread core/src/callback.jl Outdated
@visr
visr merged commit 7675bed into main Jun 3, 2025
@visr
visr deleted the transient_basin_forcing branch June 3, 2025 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cyclic_time model is not cyclic Add reduction factor to flow expressions for LinearResistance and Manningresistance in docs

2 participants