Skip to content

Tests for stereo reconstruction South#49

Merged
GernotMaier merged 5 commits intomainfrom
south-telid
Feb 17, 2026
Merged

Tests for stereo reconstruction South#49
GernotMaier merged 5 commits intomainfrom
south-telid

Conversation

@GernotMaier
Copy link
Copy Markdown
Member

No description provided.

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 updates telescope-array flattening and training data loading to better handle variable-length indexing (CTAO/ImgSel_list) and heterogeneous input files.

Changes:

  • Add padding so per-telescope arrays from different branches share a consistent width before remapping to telescope-ID space.
  • Pad size and other telescope variables to avoid width mismatches during normalization.
  • Update load_training_data() to refresh max_tel_id when later input files contain larger telescope IDs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GernotMaier GernotMaier marked this pull request as ready for review February 17, 2026 10:38
@GernotMaier GernotMaier requested a review from Copilot February 17, 2026 10:38
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

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@GernotMaier GernotMaier merged commit 7347011 into main Feb 17, 2026
2 checks passed
@GernotMaier GernotMaier deleted the south-telid branch February 17, 2026 10:46
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.

2 participants