Skip to content

Update layout names for South beta layouts in production model 7.0.0.#2164

Merged
GernotMaier merged 2 commits into
mainfrom
south-layouts
May 4, 2026
Merged

Update layout names for South beta layouts in production model 7.0.0.#2164
GernotMaier merged 2 commits into
mainfrom
south-layouts

Conversation

@GernotMaier
Copy link
Copy Markdown
Contributor

(a bit cumbersome)

@GernotMaier GernotMaier self-assigned this May 4, 2026
@GernotMaier GernotMaier added the Skip review PR without required review label May 4, 2026
@GernotMaier GernotMaier marked this pull request as ready for review May 4, 2026 17:54
@GernotMaier GernotMaier requested a review from Copilot May 4, 2026 17:54
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

Updates integration test workflow configs to use version-dependent array layout names, aligning South “alpha/beta” layouts with the renamed CTAO layout identifiers introduced for production model 7.0.0.

Changes:

  • Switched several simtools-simulate-prod integration test configs from a single array_layout_name string to a by_version mapping (<7.0.0 vs >=7.0.0).
  • Updated expected output filenames in the “multiple model versions” South gamma config (currently inconsistent with the model versions under test).
  • Added a changelog fragment documenting the layout-name update.

Reviewed changes

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

Show a summary per file
File Description
tests/integration_tests/config/simulate_prod_gamma_62_deg_south_check_output.yml Use array_layout_name.by_version to map betaCTAO-South-Beta for model_version >= 7.0.0.
tests/integration_tests/config/simulate_prod_gamma_40_deg_south_corsika_only.yml Same by_version mapping for the South beta layout.
tests/integration_tests/config/simulate_prod_gamma_40_deg_south_check_output.yml Same by_version mapping for the South beta layout.
tests/integration_tests/config/simulate_prod_gamma_20_deg_south_multiple_model_versions.yml Introduce by_version mapping for South alpha layout, but expected output filenames were changed inconsistently for 6.x model versions.
docs/changes/2164.model.md Changelog fragment for the layout-name update.

@GernotMaier GernotMaier merged commit e6836c1 into main May 4, 2026
20 checks passed
@GernotMaier GernotMaier deleted the south-layouts branch May 4, 2026 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip review PR without required review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants