Skip to content

Add CORSIKA v78050; simplified matrix#1983

Merged
GernotMaier merged 5 commits intomainfrom
CORSIKA-78050
Jan 16, 2026
Merged

Add CORSIKA v78050; simplified matrix#1983
GernotMaier merged 5 commits intomainfrom
CORSIKA-78050

Conversation

@GernotMaier
Copy link
Copy Markdown
Contributor

@GernotMaier GernotMaier commented Jan 15, 2026

Add CORSIKA v78050 to build matrix. Simplify build matrix to avoid many duplicated values.

Required new releases of

See https://github.com/gammasim/simtools/actions/runs/21033352047 for a complete build and deploy.

@GernotMaier GernotMaier self-assigned this Jan 15, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for CORSIKA v78050 and simplifies the GitHub Actions build matrix by replacing explicit version combinations with a more maintainable structure using matrix variables and includes.

Changes:

  • Added CORSIKA v78050 with corresponding configuration versions (corsika_config v1.0.0 and corsika_opt_patch v1.2.0)
  • Refactored the build matrix from explicit version objects to separate dimensions (corsika versions × avx_flag options)
  • Added changelog entry documenting the new CORSIKA version

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/changes/1983.feature.md Adds changelog entry for CORSIKA v78050 with link to release notes
.github/workflows/build-corsika7.yml Simplifies build matrix structure and adds CORSIKA v78050 configuration

GernotMaier and others added 2 commits January 15, 2026 15:57
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@GernotMaier GernotMaier marked this pull request as ready for review January 15, 2026 14:59
@ctao-sonarqube
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@GernotMaier
Copy link
Copy Markdown
Contributor Author

Thanks @tobiaskleiner

@GernotMaier GernotMaier merged commit 9d39577 into main Jan 16, 2026
16 checks passed
@GernotMaier GernotMaier deleted the CORSIKA-78050 branch January 16, 2026 07:56
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.

3 participants