Skip to content

Conversation

@kvrigor
Copy link
Member

@kvrigor kvrigor commented Nov 27, 2025

Inspired by @AGonzalezNicolas's matrix builds (#109). With this change this becomes possible:

./build_tsmp2.sh icon eclm parflow
echo "eCLM ICON ParFlow" | xargs -t ./build_tsmp2.sh --no_update

cat << EOF > model_combinations.txt
icon eclm parflow
eclm parflowgpu
icon
EOF
cat model_combinations.txt | xargs -t -L 1 ./build_tsmp2.sh --no_update

@kvrigor kvrigor changed the title build_tsmp2: Specify component model as arguments build_tsmp2: Specified component model as arguments Nov 27, 2025
@AGonzalezNicolas
Copy link
Contributor

I really like it but I am missing the use of different environments. Of course, the command can be submitted a second time with another environment provided that previous builds are moved or working in a different location. Otherwise, new build with new environment will replace previous build.

@jjokella
Copy link
Contributor

Very cool!

I have still the question of the "forced updated" (in the sense that we don't have to type any "y" or "yes").

This could maybe be an extra-option that could be added in this PR? Or we could make a new one?

@kvrigor
Copy link
Member Author

kvrigor commented Nov 27, 2025

I have still the question of the "forced updated" (in the sense that we don't have to type any "y" or "yes").

Maybe yes is the answer: yes y | ./build_tsmp2.sh

@kvrigor
Copy link
Member Author

kvrigor commented Nov 27, 2025

the command can be submitted a second time with another environment provided that previous builds are moved or working in a different location

@AGonzalezNicolas --build_dir/--install_dir for this use case:

# ICON-eCLM-ParFlow, GNU+ParaStationMPI
echo "icon eclm parflow" | xargs -t -L 1 ./build_tsmp2.sh --no_update --env env/jsc.2025.gnu.psmpi --build_dir bld/ICON_eCLM_ParFlow_GNUPSMPI --install_dir bin/ICON_eCLM_ParFlow_GNUPSMPI

# eCLM-ParFlowGPU, GNU+OpenMPI
echo "eclm parflowgpu" | xargs -t -L 1 ./build_tsmp2.sh --no_update --env env/jsc.2025.gnu.openmpi --build_dir bld/eCLM_ParFlowGPU_GNUOPENMPI --install_dir bin/eCLM_ParFlowGPU_GNUOPENMPI

@kvrigor kvrigor changed the title build_tsmp2: Specified component model as arguments build_tsmp2: Interpret component model inputs as arguments Nov 27, 2025
@kvrigor
Copy link
Member Author

kvrigor commented Nov 27, 2025

@jjokella We can address the concern for build_tsmp2 model updates (auto, forced or otherwise) on a separate GitHub issue/PR.

@kvrigor kvrigor merged commit 1563898 into master Nov 28, 2025
5 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.

5 participants