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
Change the track associator in HI Track Validation #7404
Change the track associator in HI Track Validation #7404
Conversation
A new Pull Request was created by @kkrajczar (Krisztian Krajczar) for CMSSW_7_4_X. Change the track associator in HI Track Validation It involves the following packages: Validation/RecoHI @cmsbuild, @danduggan, @nclopezo, @deguio can you please review it and eventually sign? Thanks. |
-1 - do not change parameters of existing modules - the clone is needed |
Pull request #7404 was updated. @cmsbuild, @danduggan, @nclopezo, @deguio can you please check and sign again. |
@cmsbuild please test |
Hi David, It turned out that the parameter already had the value it was changed to ('reco'), so I removed the superfluous parameter change. Now, the existing module is used as is. |
The tests are being triggered in jenkins. |
Pull request #7404 was updated. @cmsbuild, @danduggan, @nclopezo, @deguio can you please check and sign again. |
-1 runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15+MINIAODMCUP15/step2_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15+MINIAODMCUP15.log ----- Begin Fatal Exception 28-Jan-2015 19:18:51 CET----------------------- An exception of category 'Configuration' occurred while [0] Processing run: 1 lumi: 1 event: 1 [1] Running path 'digitisation_step' [2] Calling event method for module MixingModule/'mix' Exception Message: RootInputFileSequence::readOneRandom(): no input files specified for secondary input source. ----- End Fatal Exception ------------------------------------------------- you can see the results of the tests here: |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request will be automatically merged. |
Change the track associator in HI Track Validation
This PR is a workaround to the "RuntimeError: An entry in sequence tracksValidationFS has no label" error observed in recent IBs for HI Validation. That problem seems to be caused by PR #7175 (main authors are @Dr15Jones and @davidlange6). Please see other comments in that PR, PR #7175 .