-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TRestRawMultiFEMINOSToSignalProcess: first two events are with the same id #36
Comments
Looking at the code, the bug looks more or less clear to me: rawlib/src/TRestRawMultiFEMINOSToSignalProcess.cxx Lines 496 to 507 in 0838016
So I am not surprised that the timestamp and the ID are the same for the first 2 events. |
It seems the first event is stampped by the header frame after it, and the other events are stampped by the header frame ahead. So actually the first two events are using same header. What is the correct place for the header in FEMINOS daq? |
Following the FEMINOS manual the Event Count and the Timestamp are written once per event, I guess that if you have several FEMINOS you should get one Event Count and Timestamp per FEMINOS. As far as I understand, event boundaries are between Anyhow, perhaps would be better if the person that wrote the code comment on that. |
There are few comments above line 496. In principle, the code mentioned by @juanangp should only be accessed in case the event id of the event is =0. I guess this should not be the case, or it is? It is the DAQ event id starting by zero? If that lines of code are at the origin of the problem, perhaps a solution would be to initialise ID to -1, and then use the condition |
I found that the first two events from TRestRawMultiFEMINOSToSignalProcess are with same id. Don't know if is the file/daq problem or the process's problem.
I download this file: https://github.com/rest-for-physics/rawlib/blob/master/pipeline/data/R01208_Ar2Iso_Background14h_14Vetos_IccubFEC-000.aqs
and type directly in restRoot:
And see the timestamp of the first two events are same. We can also try event id, and see they are same. However, the signal data seems to be different, meaning that the events are not actually same.
The text was updated successfully, but these errors were encountered: