You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The source reuses the same container for each event, which creates the potential for hard to trace bugs (information from a previous event creeping over into the next) and even uses mutable default arguments (in ctapipe_io_lst.containers) resulting in the exact same containers being used by multiple source instances.
This comes from older versions of ctapipe, where no better solution was available and creating containers was slow, this has improved since a long time though and the same changes have been made in ctapipe.
Sources should not "reuse" containers.
The text was updated successfully, but these errors were encountered:
The source reuses the same container for each event, which creates the potential for hard to trace bugs (information from a previous event creeping over into the next) and even uses mutable default arguments (in
ctapipe_io_lst.containers
) resulting in the exact same containers being used by multiple source instances.This comes from older versions of ctapipe, where no better solution was available and creating containers was slow, this has improved since a long time though and the same changes have been made in ctapipe.
Sources should not "reuse" containers.
The text was updated successfully, but these errors were encountered: