Skip to content

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Oct 9, 2024

This speeds up the configure step from 20.1sec to 6.5sec by just downloading a shallow copy of AMReX & pybind11.

@ax3l ax3l added install Build and installation Performance optimization labels Oct 9, 2024
@ax3l
Copy link
Member Author

ax3l commented Oct 9, 2024

@EZoni @WeiqunZhang this PR is not urgent and I can take it on when I am back from parental leave. I want to introduce it consistently in pyAMReX/WarpX/ImpactX/HiPACE during the same release cycle. Can wait for 2025.

I already use the same trick in openPMD-api 0.16.0+
As an alternative approach, one could do shallow (--depth 1) git clones, but unfortunately, the CMake git fetcher does not support shallow copies.

@ax3l ax3l requested a review from EZoni October 9, 2024 18:59
@ax3l ax3l changed the title [WIP] CMake: Tarball Suppport [WIP] CMake: Tarball Support Oct 9, 2024
This speeds up the configure step from 20.1sec to 6.5sec by just
downloading a shallow copy of AMReX & pybind11.
@ax3l
Copy link
Member Author

ax3l commented Sep 8, 2025

Hi @EZoni,

Do you mind adding the WarpX Tools/Release/update_dependencies.py script (simply in /) to pyAMReX, too?
I would then improve that script in the PR here to automatically calculate checksums for the tarballs.

ax3l added a commit that referenced this pull request Sep 17, 2025
Requested by @ax3l on Slack (to be used in #378):

> Do you mind adding the Tools/Release/update_dependencies.py script
(simply in /) to pyAMReX, too? Do not need the other automations for
now.

WarpX references:
- BLAST-WarpX/warpx#5965
- BLAST-WarpX/warpx#5993
- BLAST-WarpX/warpx#6038

To-do:
- [x] AMReX
- [x] pybind11

---------

Co-authored-by: Axel Huebl <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant