SiWECAL-TestBeam / SiWECAL-TB-analysis

Scripts for SiW ECAL test beam analysis
GNU General Public License v3.0
2 stars 7 forks source link

possible mixing of cycleIDs in the raw data decoder #31

Open airqui opened 2 years ago

airqui commented 2 years ago

Discussing with Jihane we realized that there is not prohibited that the raw data frames come unsorted in cycleID. This could happen if we have very large events (all sca filled) and somehow different occupancy in the slabs (i.e. one slab super noisy and others very quiet).

This will require a deep revision of the raw converter (for ASCII and binary). To be done after the beam test. My suspicion is that for ASCII may be more severe effect because the data writing is actually much slower. For the binary files, it seems to not be a problem:

For the time being, I propose to simply "tag" the cycleIDs with potential issues. For that, I attached the TH1F wrongly_reconstructed_cycles with the data.

This histogram has to be used to veto cycleIDs

airqui commented 2 years ago

Problem addressed in https://github.com/SiWECAL-TestBeam/SiWECAL-TB-analysis/pull/33 Only for the binary data.