Add self-bundling to ecTrans#399
Merged
samhatfield merged 3 commits intoMay 11, 2026
Merged
Conversation
Collaborator
|
Nice to see this, I've been putting this off for years. After this PR we will have two arch directories (https://github.com/ecmwf-ifs/ectrans/tree/develop/.github/arch). So let's either combine the two in this PR, or sort it out afterwards with another PR. |
samhatfield
reviewed
May 8, 2026
samhatfield
reviewed
May 8, 2026
samhatfield
reviewed
May 8, 2026
samhatfield
reviewed
May 8, 2026
samhatfield
reviewed
May 8, 2026
Comment on lines
+49
to
+71
| ## bundle build, text to be moved to sites.ecmwf.int/[]/installation.html | ||
| Build using ecTrans bundle | ||
|
|
||
| Another way of building ecTrans is to use the bundle definition included in `package/bundle`: | ||
|
|
||
| $ ./package/bundle/ectrans-bundle create --bundle package/bundle/bundle.yml # Checks out dependency packages into "source" directory | ||
| $ ./package/bundle/ectrans-bundle build [--build-type=<build-type>] [--arch=<path-to-arch>] [--option] | ||
|
|
||
| The bundle also facilitates setting environment variables and compiler flags relevant to certain architectures by specifying the corresponding arch file at the build step. For example, to build on the ECMWF Atos system using Intel compilers and the hpcx-openmpi `MPI` library: | ||
|
|
||
| `--arch=package/bundle/arch/ecmwf/hpc2020/intel/2021.4.0/hpcx-openmpi/2.9.0` | ||
| or equivalently | ||
| `--arch ecmwf/hpc2020/intel/2021.4.0/hpcx-openmpi/2.9.0` | ||
|
|
||
| A number of options can also be configured during the bundle build step, including: | ||
| - `--without-mpi` - Disable MPI | ||
| - `--without-single-precision` - Only build double-precision variant of ecTrans | ||
| All options can be seen at the end of the `package/bundle/bundle.yml` file. | ||
|
|
||
| Finally, additional `CMake` options can also be set during the bundle build step: | ||
|
|
||
| `--cmake="OPTION=<arg>"` | ||
|
|
Collaborator
There was a problem hiding this comment.
Will put this on the docs site so you can remove it here.
Collaborator
Author
|
Comments addressed, thanks for the review. I suggest merging the two arch locations in a subsequent PR. |
Collaborator
|
Nice I'll play around with this next week! |
samhatfield
added a commit
that referenced
this pull request
May 11, 2026
samhatfield
approved these changes
May 11, 2026
wdeconinck
added a commit
to dhaumont/ectrans
that referenced
this pull request
May 15, 2026
* develop: Fixup and simplify toolchains (ecmwf-ifs#401) Make ci of nvhpc with github-hosted runners use "-tp=haswell" flag (ecmwf-ifs#400) Correct name of nvidia arch Consolidate arch directories Add self-bundling to ecTrans (ecmwf-ifs#399) Implement new IFS compile flags convention (ecmwf-ifs#397) Remove compile flag overloads for ftinv/ftdir GPU (ecmwf-ifs#396) Remove legacy compiler workarounds (ecmwf-ifs#393) Fix ci failures with nvhpc and intel (ecmwf-ifs#394) Make adjoint utils library platform dependent (ecmwf-ifs#390) Make test ectrans_test_split_mpi_comm dependent on fiat version 2.0 Fix compilation warning in tests/trans/api/fpe_trapping
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add ectrans-bundle to the ecTrans repo.
Contributor Declaration
By opening this pull request, I affirm the following: