New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add fields defined in CTAO DL0 and R1 data models #2338
Conversation
ccf3df5
to
2cab3be
Compare
ed93d63
to
9d2754d
Compare
# unset dvr bits for removed pixels | ||
dl0_pixel_status[~signal_pixels] &= ~np.uint8(PixelStatus.DVR_STATUS) | ||
|
||
event.dl0.tel[tel_id] = DL0CameraContainer( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a somewhat unrelated improvement. Instead of relying on that the first access of a non-existing container will default initialize it (as Map
behaves like a defaultdict
), we explicitly create the new container.
This is more efficient (since the defaults don't have to be created before they are overwritten with the actual values) and less error prone (because you can't by accident "inherit" values from before).
See some discussion in #2084
Pixel status information | ||
|
||
See DL0 Data Model specification: | ||
https://redmine.cta-observatory.org/dmsf/files/17552/view |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most people developing ctapipe will have access to it and it is the official source for these definitions. I think not including it for reference is worse.
No description provided.