Skip to content

Conversation

@ax3l
Copy link
Member

@ax3l ax3l commented Dec 8, 2025

Can be merged before the WIP PR in #6396 #6383

  • Azure Runners update w/ CCache acceleration
  • Pass-Through WarpX_PETSC option

Follow-up PR:

  • Tuo (LLNL): PETSc w/ SuperLU

Build-only runner.
@ax3l ax3l added component: tests Tests and CI install component: third party Changes in WarpX that reflect a change in a third-party library labels Dec 8, 2025
@ax3l ax3l marked this pull request as ready for review December 8, 2025 22:12
@ax3l ax3l requested a review from RemiLehe December 8, 2025 22:30
@ax3l ax3l requested a review from debog December 8, 2025 22:32
@ax3l ax3l changed the title PETSC: Better Support in Build & CI PETSc: Better Support in Build & CI Dec 8, 2025
Comment on lines 192 to 193
--download-superlu \
--download-superlu_dist \
Copy link
Member Author

@ax3l ax3l Dec 8, 2025

Choose a reason for hiding this comment

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

SuperLU install fails on Tuo:

...
=============================================================================================
              Trying to download https://github.com/xiaoyeli/superlu for SUPERLU
=============================================================================================
=============================================================================================
                Configuring SUPERLU with CMake; this may take several minutes
=============================================================================================
=============================================================================================
               Compiling and installing SUPERLU; this may take several minutes
=============================================================================================
=============================================================================================
         Trying to download https://github.com/xiaoyeli/superlu_dist for SUPERLU_DIST
=============================================================================================
=============================================================================================
              Configuring SUPERLU_DIST with CMake; this may take several minutes
=============================================================================================
=============================================================================================
             Compiling and installing SUPERLU_DIST; this may take several minutes
=============================================================================================

*********************************************************************************************
           UNABLE to CONFIGURE with GIVEN OPTIONS (see configure.log for details):
---------------------------------------------------------------------------------------------
                             Error running make on  SUPERLU_DIST
*********************************************************************************************

Copy link
Member Author

Choose a reason for hiding this comment

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

Can go in a follow-up PR, something machine specific.

Pass-through control for pre-built and superbuild AMReX.
@ax3l ax3l force-pushed the ci-petsc-azure branch 7 times, most recently from eafe289 to 6bdd2cb Compare December 10, 2025 18:27
@ax3l
Copy link
Member Author

ax3l commented Dec 18, 2025

@JustinRayAngus @debog @EZoni this one is ready for merge.

Copy link
Contributor

@JustinRayAngus JustinRayAngus left a comment

Choose a reason for hiding this comment

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

Thanks!

@JustinRayAngus JustinRayAngus merged commit 75888ad into BLAST-WarpX:development Dec 18, 2025
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: tests Tests and CI component: third party Changes in WarpX that reflect a change in a third-party library install

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants