Skip to content

SOLVERS: prefer cfdemSolverPimple over cfdemSolverPiso by default - #3

Open
marniemann wants to merge 1 commit into
mainfrom
solvers-prefer-pimple
Open

marniemann wants to merge 1 commit into
mainfrom
solvers-prefer-pimple

Conversation

@marniemann

Copy link
Copy Markdown
Contributor

Summary

  • Flips the default solver recommendation: cfdemSolverPimple should be preferred over cfdemSolverPiso even for laminar/single-outer-corrector cases — it runs safely in Piso-equivalent mode (nOuterCorrectors 1) with no performance penalty, is generally more accurate for coupled CFD-DEM, and exposes more features (turbulence, smoothing models, expCorrDeltaUError) if the case grows into needing them.
  • cfdemSolverPiso is now documented as the fallback for installs that don't build cfdemSolverPimple, with a check (ls $CFDEM_APP_DIR | grep cfdemSolver / which cfdemSolverPimple) for confirming availability before assuming it's present.

Test plan

  • Confirm the updated guidance matches current internal recommendations
  • Verify wording doesn't need a similar update anywhere else referencing solver choice

Pimple runs safely in Piso-equivalent mode (nOuterCorrectors 1) with
no performance penalty, is generally more accurate for coupled CFD-DEM,
and exposes more features. Piso remains the fallback for installs that
don't build Pimple.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@marniemann
marniemann requested a review from d-louw September 18, 2026 09:42
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.

1 participant