Skip to content

Conversation

baperry2
Copy link
Contributor

@baperry2 baperry2 commented Jul 17, 2025

We rebuild PelePhysics over and over in the CI to test various different mechanisms, EOS, and Transport models. We can be more efficient by building AMReX as a library once and linking to it, instead of rebuilding all the AMReX stuff every time we rebuild PelePhysics.

In addition, this PR makes a few changes to promote faster results in the CI:

  • Don't run the free disk space script, which takes a nontrivial amount of time and doesn't seem to be necessary
  • Move some stuff specific to the Jacobian test from the dependencies section to get to actual tests faster

TODO:

  • @jrood-nrel : right now the AMReX build (as well as the Sundials build) don't use ccache, perhaps they could benefit from that as well?
  • There are some errors in the ReactEval case, which I converted from 3D to 2D to only have to use one version of AMReX. That shouldn't affect the results, so more investigation is needed here

@baperry2 baperry2 marked this pull request as ready for review July 18, 2025 00:43
@baperry2 baperry2 requested a review from jrood-nrel July 18, 2025 00:43
@baperry2 baperry2 enabled auto-merge (squash) July 29, 2025 15:45
@baperry2 baperry2 merged commit 69ce07f into AMReX-Combustion:development Jul 29, 2025
9 checks passed
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.

2 participants