Skip to content

remove newaxis#265

Merged
TjarkMiener merged 2 commits into
mainfrom
fix_ctl_tels
Oct 23, 2025
Merged

remove newaxis#265
TjarkMiener merged 2 commits into
mainfrom
fix_ctl_tels

Conversation

@BastienLacave
Copy link
Copy Markdown
Collaborator

Write CTLearn_telescopes in a column instead of in a row (bug), this divides the file size by 1000. Anly for predict LST1 tool

@TjarkMiener
Copy link
Copy Markdown
Member

Thanks @BastienLacave! is_valid is a simple boolean, but ..._telescopes should be an array of boolean, where each entry has a boolean array of length n_tels. Here n_tels is 1 since we only have LST-1. I think the problem is that [np.newaxis] is makinig the dimension over the wrong axis. Can you please check?

@TjarkMiener TjarkMiener merged commit acb7eca into main Oct 23, 2025
5 of 9 checks passed
@TjarkMiener TjarkMiener deleted the fix_ctl_tels branch October 23, 2025 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants