Tighten Ds*, K*Gamma and PhiGamma and add new line Eta2MuMuGamma
I tried the following modifications to the CaloPID lines:
-
include the missing stripping cuts on the Ds, for that I added a new Filter that applies to the Hlt2Shared_Ds;
-
use TightKstar2KPi and UnbiasedPhi2KK in place of Kstar2KPi and Phi2KK for the KGamma and PhiGamma lines. In that way, I only modified Inputs.py and not the Combiners in Stages.py. The following additional cuts are now applied: PIDK(K)>0, PT(K+,pi-)>500 MeV and mass window from 300 to 100 MeV for the K and from 50 to 30 for the Phi. Looking at the offline cuts used for the Run1 PID samples, these are all used, so I think it is OK.
The Moore rate test results using v26r5p1 are:
-
from 30 to 10 Hz for the Ds* line;
-
from 10 to <10 Hz for the K*Gamma;
-
from <10 to <10 Hz for the PhiGamma;
So we saved 20-30 Hz which can be used to either lower the PT cut of the Ds* line or include the Eta2MuMuGamma line. (RM: the latter was done)
Log files with old and new settings are attached.
Merge request reports
Activity
I tried the following modifications to the CaloPID lines:
-
include the missing stripping cuts on the Ds, for that I added a new Filter that applies to the Hlt2Shared_Ds;
-
use TightKstar2KPi and UnbiasedPhi2KK in place of Kstar2KPi and Phi2KK for the KGamma and PhiGamma lines. In that way, I only modified Inputs.py and not the Combiners in Stages.py. The following additional cuts are now applied: PIDK(K)>0, PT(K+,pi-)>500 MeV and mass window from 300 to 100 MeV for the K and from 50 to 30 for the Phi. Looking at the offline cuts used for the Run1 PID samples, these are all used, so I think it is OK.
Edited by Maximilien Chefdeville-
- Resolved by Rosen Matev
Then we might want to move it to
Hlt2SharedParticles
but I leave that question to the Hlt team :) The rest LGTM.About the issue with the rate test, I did some checks and my conclusion is that the problem arises when the test is run with NO
PersistReco
line. As soon as you add one (Exotica
is an small module with at leat onePersistReco
line) it runs fine. This happens also withMoore v25r4
but did not happen withMoore v25r1
. For completeness, the error one gets if running only theCaloPID
module is:HltPersistRecoClonerSequence INFO Member list: GaudiSequencer/HltFullPersistRecoSequence, Gaudi Sequencer/HltCopyTurboOutputsSequence JobOptionsSvc ERROR Unable to set the property 'Members' of 'HltFullPersistRecoSequence'. Check option and algorithm names, type and bounds. HltPersistRecoClonerSequence ERROR GaudiSequencer:: Can not initialize HltFullPersistRecoSequence StatusCode=FAILURE HltPersistRecoSequence ERROR GaudiSequencer:: Can not initialize HltPersistRecoClonerSequence StatusCode=FAILURE HltAfterburnerSequence ERROR GaudiSequencer:: Can not initialize HltPersistRecoSequence StatusCode=FAILURE HltAfterburnerFilterSequence ERROR GaudiSequencer:: Can not initialize HltAfterburnerSequence StatusCode=FAILURE HltAfterburner ERROR GaudiSequencer:: Can not initialize HltAfterburnerFilterSequence StatusCode=FAILURE HltDecisionSequence ERROR GaudiSequencer:: Can not initialize HltAfterburner StatusCode=FAILURE Hlt ERROR GaudiSequencer:: Can not initialize HltDecisionSequence StatusCode=FAILURE EventLoopMgr ERROR Unable to initialize Algorithm: Hlt ServiceManager ERROR Unable to initialize Service: EventLoopMgr ServiceManager ERROR Service EventLoopMgr not in the correct state to be initialized (CONFIGURED) ServiceManager ERROR Service EventLoopMgr not in the correct state to be initialized (CONFIGURED) ApplicationMgr FATAL stop: Invalid state "CONFIGURED"
- [2017-08-26 00:07] Validation started with lhcb-2017-patches#121
- [2017-08-27 00:06] Validation started with lhcb-2017-patches#122
- [2017-08-28 00:06] Validation started with lhcb-2017-patches#123
- [2017-08-29 00:05] Validation started with lhcb-2017-patches#124
- [2017-08-30 00:06] Validation started with lhcb-2017-patches#125
- [2017-08-31 00:05] Validation started with lhcb-2017-patches#126
- [2017-09-01 00:05] Validation started with lhcb-2017-patches#127
- [2017-09-02 00:06] Validation started with lhcb-2017-patches#128
- [2017-09-03 00:07] Validation started with lhcb-2017-patches#129
- [2017-09-04 00:07] Validation started with lhcb-2017-patches#130
Edited by Software for LHCbNew TURCAL line using DiMuon trigger. Largely based on Hlt2ExoticaPrmptDiMuonTurbo:
- DiMuonNoIPCombiner;
- PrmptDiMuonFilter;
then add gamma with PT>500 MeV:
- Eta2MuMuGammaCombiner.
Rate tests on 100k events:
- 30 Hz with PT>200 MeV (test_eta2mumugamma_100k_200mev.log);
- 20 Hz with PT>500 MeV (test_eta2mumugamma_100k_500mev.log).
Edited by Maximilien Chefdeville
added ~1529 line ~1530 labels
changed milestone to %v26r6
assigned to @rmatev
mentioned in commit e16771f7