Refactoring the image building process (CORSIKA/sim_telarray/simtools)#1917
Merged
GernotMaier merged 90 commits intomainfrom Dec 16, 2025
Merged
Refactoring the image building process (CORSIKA/sim_telarray/simtools)#1917GernotMaier merged 90 commits intomainfrom
GernotMaier merged 90 commits intomainfrom
Conversation
of a corsika-only docker
…jection Integration tests issues with new docker files...
orelgueta
reviewed
Dec 9, 2025
Contributor
orelgueta
left a comment
There was a problem hiding this comment.
This all looks great. I only made a few comments/questions/suggestions, but none of them is critical.
Contributor
Author
|
Thanks @orelgueta - I've added now everything or moved tasks into issues. Tests / linters fail due to #1941 and need #1939 to be merged - we can wait for that if needed. Let me know if there is anything else I can do. |
orelgueta
approved these changes
Dec 10, 2025
Contributor
orelgueta
left a comment
There was a problem hiding this comment.
All good, you can merge before or after fixing the tests, I don't mind.
|
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.




Refactoring the image building process (CORSIKA/sim_telarray/simtools):
/workdir/simulation_software/with e.g./workdir/simulation_software/corsika/workdir/simulation_software/sim_telarrayNew Docker files / workflows:
docker/Dockerfile-corsika7,.github/workflows/build-corsika7.yml: build CORSIKA7 for different opimization using KIT gitlab (no QGSTables included)docker/Dockerfile-simtel_array,.github/workflows/build-sim_telarray.yml: build sim_telarray, hessioxxx, stdools using CTAO gitlab. No CORSIKA included (temporary: hessioxxx from tar package)docker/Dockerfile-simtools-dev,.github/workflows/build-simtools-dev.yml: develop image for simtools include the images build in the steps 1 and 2; QGSJet-III Table added;no_optbuild.docker/Dockerfile-simtools-prod,.github/workflows/build-simtools-prod.yml(missing):productions images for simtools (several images for each optimization step; includes images from steps 1 and 2)Legacy Docker files :
docker/Dockerfile-corsika-simtel,.github/workflows/build-corsika-simtel.yml: build CORSIKA and sim_telarray using tar packagesdocker/Dockerfile-simtools-legacy,.github/workflows/build-simtools-legacy.yml: simtools production image (includes qgs tables) based on Dockerfile-corsika-simtel imagesOpen points:
EPOS executable is included, but corsika_autoinputs probably needs an update to get it working (issue #1925 )