Make is_simulated a property of the event source

[maxnoe]

I think this is important information that should not be hidden in metadata and be required to be implemented for all event sources.

Opinions?

[vuillaut]

I agree.

[watsonjj]

I have nothing against it. But we only really have one simulation source, so its quite clear already. Did you encounter a particular need for this information, other than for "metadata" reasons?

[maxnoe]

I don't really want to see isinstance(event_source, SimTelEventSource) when it could be
if event_source.is_simulated.

[watsonjj]

But what follows that if statement that is guaranteed to be valid for any simulation event source?

[maxnoe]

Yes, e.g. writing the mc container to the DL1 file.

[watsonjj]

Okay, that sounds valid (assuming any simulation event source is guaranteed to have implemented that container)

[kosack]

Sounds good to me. You might also want some other meta info like what data level the event source provides.

print(eventsource.is_simulated).
print(eventsource.data_levels) 

True
['R0', 'R1']

Where eventsource.data_level is a list of what data levels are generated:

• ["DL1",] for a DL1EventSource, when that exists
• ["R0","R1"] for SimTelEventSource

[maxnoe]

@kosack that's a good idea, will include that