EventSource.subarray must be a property
maxnoe opened this issue · comments
Maximilian Linhoff commented
Currently the lst event source deviates from the API of the baseclass, which makes it impossible to use in programs that just use the generic EventSource
and rely on a common API.
Franca Cassol commented
Hi @maxnoe,
yes, I think the implementation of the subarray was not orthodox, could you indicate a source with the correct API, to take as example?
Maximilian Linhoff commented
ctapipe.io.EventSource
is the abstract baseclass, which has the API we need to implement.
.subarray
needs to be a simple property, so one can do:
event_source = ...
print(event_source.subarray)