-
Notifications
You must be signed in to change notification settings - Fork 444
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
O2-4592 - adding TOF LTIntegral to time series&skimmed data #13809
base: dev
Are you sure you want to change the base?
Conversation
REQUEST FOR PRODUCTION RELEASES:
This will add The following labels are available |
Hello @noferini , @ercolessi @matthias-kleiner As we agreed I am pushing the TOF information into the TPC time series & skimmed data. New time series:
The content looks reasonable so far:
|
@miranov25 you have to apply |
AddingTPC, Vertex, and TPC-TOF time
|
@shahor02, @davidrohr, @wiechula Could you please review the pull request? I would like to request a cherry-pick of the code. Local tests have been successful without any issues. Best regards, |
Somehow clang is stuck, for a moment. I will add it clang version soon.
|
Hi @miranov25 ,
|
Hello @shahor02, @noferini, and @wiechula, I ran
This suggests there might be a bug in the build system. Could this be approved so I can request a cherry-pick for the apass1 production? Data Volume for mTRegarding the data volume for the mT, I will apply compression with an estimated factor of 4–8. It is straightforward since we only need to encode very small deltas relative to the expectation:
Here:
I will review how this is currently implemented in the AO2D, but it can be coded efficiently in a similar manner. I am prototyping the approach now. Best regards, |
Error while checking build/O2/fullCI for 1bc9730 at 2024-12-17 09:05:
Full log here. |
Adding TOF information structure o2::track::TrackLTIntegral ltIntegral=tpctofmatch.getLTIntegralOut() into the