search

cta-observatory / ctapipe

Low-level data processing pipeline software for CTAO or similar arrays of Imaging Atmospheric Cherenkov Telescopes
https://ctapipe.readthedocs.org
BSD 3-Clause "New" or "Revised" License
64 stars 268 forks source link

Fix HDF5EventSource.__len__ if allowed_tels is not None #2325

Open maxnoe opened 1 year ago

maxnoe commented 1 year ago

One of the possible solutions for #2324, however, this performs a potentially expensive operation just to get the correct __len__, which is currently only used to provide a nice progress bar, so it might be better just to remove __len__.