Uh oh!
There was an error while loading. Please reload this page.
AOD stores tracks parameters at calorimeters - #10611
Conversation
Error while checking build/O2/fullCI for 1715391 at 2023-01-25 15:04: Full log here. |
ktf
commented
Jan 23, 2023
@shahor02@davidrohr@sawenzel all the PRs are now failing with: in the fullCI. Do you have any idea of what is going on? |
shahor02
commented
Jan 23, 2023
Yes, I've seen this but could not reproduce it locally. I asked @sawenzel , he thinks might be resource problem. |
ktf
commented
Jan 23, 2023
I can confirm they get killed by the OOM reaper. Not yet sure why. Maybe related to the increase we also see with some analysis tasks? Did anything particularly relevant change in the last few days? |
peressounko
commented
Jan 24, 2023
Dear @shahor02, approach looks reasonable, will test on real/MC AODs once will be committed. |
mfasDa
commented
Jan 25, 2023
One thing a bit unfortunate is that the detectors use different radii for propagation (PHOS 460 cm, EMCAL 440 cm). In the overlap region between DCAL and PHOS how can we figure out which track was propagated to which detector? |
shahor02
commented
Jan 25, 2023
The |
mfasDa
commented
Jan 25, 2023
For the eta I was expecting this, and we can deal here with the eta cut in the correction task as you applied in the track extrapolation. I was more worried about the the two small DCAL supermodules in case the tracks are sufficiently curved. However the effect here is probably small. |
peressounko
commented
Jan 25, 2023
shahor02
commented
Jan 25, 2023
Such tracks are certainly possible but most of such tracks are the secondaries created in the TPC and only a fraction of them will be reconstructible in the TPC. If they are matched in TRD or TOF, then one can trust more or less their eta (though such matches will have a lot of fakes), if they are not matched (and low-pT tracks usually do not produce TRD tracklets) then the time/Z assignment of such tracks is purely conventional so their tagging for being in the EMCAL/PHOS acceptance is anyway random. |
shahor02
commented
Jan 25, 2023
@mfasDa thanks for spotting the problem with the sector definition, there are too many misleading pictures around... Fixed. |
Error while checking build/O2/fullCI for b04a04e at 2023-01-26 10:08: Full log here. |
shahor02
commented
Jan 28, 2023
@peressounko@mfasDa shall I merge this? |
peressounko
left a comment
There was a problem hiding this comment.
Looks fine, lets test with real data and MC
mfasDa
commented
Jan 29, 2023
Hi @shahor02 , I am also fine with merging! |
@mfasDa@peressounko
Please check. What it does is:
If not failed on any of the previous stages, assigns position eta and phi to already existing trackEta parameters fTrackEtaEMCAL and fTrackPhiEMCAL. Otherwise -999 is assigned to both parameters.