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
HeavyIons: switch photon tracking isolation collection to hiGeneralTracks #7374
HeavyIons: switch photon tracking isolation collection to hiGeneralTracks #7374
Conversation
A new Pull Request was created by @richard-cms (R. Alex Barbieri) for CMSSW_7_4_X. HeavyIons: switch photon tracking isolation collection to hiGeneralTracks It involves the following packages: RecoHI/HiEgammaAlgos @cmsbuild, @cvuosalo, @nclopezo, @slava77 can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
-1 runTheMatrix-results/5.1_TTbar+TTbarFS+HARVESTFS/step1_TTbar+TTbarFS+HARVESTFS.log ----- Begin Fatal Exception 27-Jan-2015 12:00:26 CET----------------------- An exception of category 'ProductNotFound' occurred while [0] Processing run: 1 lumi: 1 event: 1 [1] Running path 'simulation_step' [2] Calling event method for module HBHEIsolatedNoiseReflagger/'hbhereco' Exception Message: Principal::getByToken: Found zero products matching all criteria Looking for type: std::vector Looking for module label: trackExtrapolator Looking for productInstanceName: Additional Info: [a] If you wish to continue processing events after a ProductNotFound exception, add "SkipEvent = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration. ----- End Fatal Exception ------------------------------------------------- 135.4 step1 runTheMatrix-results/135.4_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS/step1_ZEE_13+ZEEFS_13+HARVESTUP15FS+MINIAODMCUP15FS.log ----- Begin Fatal Exception 27-Jan-2015 12:01:19 CET----------------------- An exception of category 'ProductNotFound' occurred while [0] Processing run: 1 lumi: 1 event: 1 [1] Running path 'simulation_step' [2] Calling event method for module HBHEIsolatedNoiseReflagger/'hbhereco' Exception Message: Principal::getByToken: Found zero products matching all criteria Looking for type: std::vector Looking for module label: trackExtrapolator Looking for productInstanceName: Additional Info: [a] If you wish to continue processing events after a ProductNotFound exception, add "SkipEvent = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration. ----- End Fatal Exception ------------------------------------------------- you can see the results of the tests here: |
This failure is very very surprising to me since it appears to be in a pp workflow. The HI workflow tests ran okay - I will double check why a change to the HI-specific isolation parameters made HBHE reco fail in a pp workflow, but does anyone else have any idea what happened? |
please test |
@cmsbuild please test |
The tests are being triggered in jenkins. |
.. I'm checking this |
+1 for #7374 2bd84f0 checked on 200 events of UPC data in wflow 140.53. The most relevant CPU change is an increase in the photons, as expected
|
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. |
HeavyIons: switch photon tracking isolation collection to hiGeneralTracks
With the 73X reconstruction improvements to HI, photon tracker isolation should be computed using the new tracking collection.
The previous tracking collection (hiGoodTracks) does not even exist anymore, so tracking isolation DQM histograms were just empty and we were re-running the isolation calculation at analysis level.