Do not presecal ITS reco in FST unless FST_PRESCALE_ITS defined#12528
Do not presecal ITS reco in FST unless FST_PRESCALE_ITS defined#12528shahor02 merged 1 commit intoAliceO2Group:devfrom
Conversation
|
Error while checking build/O2/fullCI for 0652d60 at 2024-01-15 22:47: Full log here. |
0652d60 to
ac1eaf4
Compare
| trackParams[0].MinTrackLength = 4; | ||
| trackParams[0].CellsPerClusterLimit = 1.e2f; | ||
| trackParams[0].TrackletsPerClusterLimit = 1.e2f; | ||
| LOG(info) << "Initializing tracker in sync. phase reconstruction with " << trackParams.size() << " passes"; |
There was a problem hiding this comment.
@mconcas @mpuccio After suppressing ITS reco prescaling in the sync. mode of FST, I got a message
Too many tracklets per cluster (3.18), check the detector status and/or the selections. Current limit is 2.0
To avoid this, I am setting the thresholds to higher values, still 10 times less than in the async mode. Please confirm that this is ok.
There was a problem hiding this comment.
@mconcas @mpuccio After suppressing ITS reco prescaling in the sync. mode of FST, I got a message
Too many tracklets per cluster (3.18), check the detector status and/or the selections. Current limit is 2.0To avoid this, I am setting the thresholds to higher values, still 10 times less than in the async mode. Please confirm that this is ok.
Changing that value would change the production setting we have in place for sync mode.
We are probably going to be safe anyway because of the prescaler, but we could think of letting the default as it is and overriding the configuration in FST with the higher value.
|
@shahor02 is something like the combination of the two suggestions below an option? I cannot test right now, sorry. |
|
Error while checking build/O2/fullCI for ac1eaf4 at 2024-01-16 09:36: Full log here. |
ac1eaf4 to
f0c5612
Compare
|
OK, I've moved increasing the thresholds to FST. |
|
Error while checking build/O2/fullCI for f0c5612 at 2024-01-16 15:04: Full log here. |
|
tested locally, merging |
No description provided.