introducing AlcaRecos in 2017 WFs #16168
Conversation
|
A new Pull Request was created by @boudoul (boudoul) for CMSSW_8_1_X. It involves the following packages: Configuration/PyReleaseValidation @cmsbuild, @srimanob, @davidlange6, @hengne, @fabozzi can you please review it and eventually sign? Thanks. cms-bot commands are list here #13028 |
|
please test |
|
The tests are being triggered in jenkins. |
|
about:
needs an update of the phase-I GT, but it's a level of sophistication we don't need at this moment in time. |
| @@ -1765,6 +1766,20 @@ def lhegensim(fragment,howMuch): | |||
| upgradeStepDict['HARVESTFull_trackingOnlyPU'][k]=merge([PUDataSets[k2],upgradeStepDict['HARVESTFull_trackingOnly'][k]]) | |||
| upgradeStepDict['HARVESTFullGlobalPU'][k]=merge([PUDataSets[k2],upgradeStepDict['HARVESTFullGlobal'][k]]) | |||
|
|
|||
|
|
|||
|
|
|||
| upgradeStepDict['ALCAFull'][k] = {'-s':'ALCA:TkAlMuonIsolated+TkAlMinBias+MuAlOverlaps+EcalESAlign', | |||
There was a problem hiding this comment.
It will be nice to add HcalCalHBHEMuonFilter to this
There was a problem hiding this comment.
@boudoul would you mind adding the request from Sunanda? as I understand this doens't lie anymore in the critical path for pre13..
There was a problem hiding this comment.
no problem, will update later tonight - thanks for the prompt feedback !
There was a problem hiding this comment.
This PR has been merged before I could modified it, so the request for the new alcareco is addressed in a new PR #16183
|
thanks @boudoul ! |
|
+1 |
cmsbuild
added
fully-signed
pdmv-approved
and removed
pdmv-pending
pending-signatures
labels
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar |
|
+1 |
This PR aims to introduce the Alcareco steps (as in run 2 MC) .
It is important to introduce the alcareco in view of MC prodcution, in particular for alignment
One comment :
There is one difference with respect to the alca steps for run2 Wfs .
I removed EcalTrg from the alca path which is crashing - Maybe this can be looked later on.