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
correct some InputTags for track collections #8866
Conversation
lveldere
commented
Apr 24, 2015
- replaced some inputtags for the non-existing collection "hltIter2HighPtTkMuMerged" with "generalTracks"
- grepped through all Famos menus for "hltIter2HighPtTkMuMerged" and corrected accordingly
- updated confdb.py accordingly
- tested with runTheMatrix -l 135.1
please test |
The tests are being triggered in jenkins. |
A new Pull Request was created by @lveldere for CMSSW_7_5_X. correct some InputTags for track collections It involves the following packages: HLTrigger/Configuration @Martin-Grunewald, @perrotta, @cmsbuild, @nclopezo, @fwyzard can you please review it and eventually sign? Thanks. |
-1 |
As you wish. |
-1 runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15+MINIAODMCUP15/step3_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15+MINIAODMCUP15.log 1330.0 step3 runTheMatrix-results/1330.0_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15+MINIAODMCUP15/step3_ZMM_13+ZMM_13+DIGIUP15+RECOUP15+HARVESTUP15+MINIAODMCUP15.log 25202.0 step3 runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25+MINIAODMCUP15/step3_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15_PU25+MINIAODMCUP15.log you can see the results of the tests here: |
@@ -456,6 +456,7 @@ def fixForFastSim(self): | |||
self._fix_parameter(name = 'src', type = 'InputTag', value = 'hltHcalTowerNoiseCleaner', replace = 'hltTowerMakerForAll') | |||
|
|||
# input tags for track collections | |||
self._fix_parameter( type = 'InputTag', value = 'hltIter2HighPtTkMuMerged', replace = 'generalTracks') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is already there (few lines down).
Please see #8867 |
@Martin-Grunewald |
@lveldere |