Skip to content
Snippets Groups Projects

Refactor CFL parameter usage

Merged Florian Atteneder requested to merge fa/refactor-cfl into main

Previously we required each project to apply CFL in their timestep! function.

We now apply CFL to the result proposed by timestep! in a central place, hence, timestep! no longer has to account for that.

This PR also moves the parameter cfl from the Mesh section to the Evolution section. All corresponding reference tests are updated here too.

Merge request reports

Merge request pipeline #6036 passed

Merge request pipeline passed for 191b4598

Approval is optional

Merged by Florian AttenederFlorian Atteneder 1 year ago (Dec 5, 2023 6:54pm UTC)

Merge details

  • Changes merged into main with 5ad99e58 (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #6037 passed

Pipeline passed for 5ad99e58 on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading