Skip to content

Add example of limiting particle timestep by local cfl in schism tuto… - #3016

Open
fluidnumericsJoe wants to merge 2 commits into
mainfrom
feature/cfl-limited-dt-tutorial-update
Open

fluidnumericsJoe wants to merge 2 commits into
mainfrom
feature/cfl-limited-dt-tutorial-update

Conversation

@fluidnumericsJoe

Copy link
Copy Markdown
Contributor

Description

This PR adds an example of limiting particle timestep by local cfl in schism tutorial

Checklist

  • This PR targets the correct branch (main for normal development, v3-support for v3 support)

@erikvansebille erikvansebille left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this extra context, @fluidnumericsJoe! Below a few small comments (I can't put comments directly on the notebook in GitHub)

  1. Is "altitude" the right word in the comment of cell [5]? That to me suggests (3D) height?
  2. Avoid variable names like h, e and p in that same cell - make them more explicit so users understand what's going on?
  3. CFL number hasn't been introduced in cell [5]. Could you move that whole cell down to the section of cell [9]? That will also make it easier to refer to this part of the tutorial directly, because all information is in one subsection
  4. Make explicit that the math symbol $C$ is the same as CFL_target
  5. Should we add a link to the CFL-section on the tutorial_dt_integrators too?

@github-project-automation github-project-automation Bot moved this from Backlog to Ready in Parcels development Oct 28, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready

Development

Successfully merging this pull request may close these issues.

2 participants