Skip to content
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.

Sign up for GitHub

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

Jump to bottom

cerr "did not find any valid" in RecoJets/JetProducers/src/PileupJetIdAlgo.cc #55

Closed
slava77 opened this issue Nov 20, 2014 · 2 comments
Closed

cerr "did not find any valid" in RecoJets/JetProducers/src/PileupJetIdAlgo.cc #55

slava77 opened this issue Nov 20, 2014 · 2 comments

Comments

@slava77
Copy link
Owner

slava77 commented Nov 20, 2014

two problems:
https://github.com/cms-sw/cmssw/blob/CMSSW_7_3_0_pre2/RecoJets/JetProducers/src/PileupJetIdAlgo.cc#L428

  • try-catch which should not be happening (the code should be changed to not use exceptions)
  • and the cerr use itself this has to be changed to LogInfo or something. This error shows up in many workflows
@slava77
Copy link
Owner Author

slava77 commented Dec 3, 2014

this replaces #34

@slava77 slava77 mentioned this issue Dec 3, 2014
Closed
@slava77
Copy link
Owner Author

slava77 commented Jun 27, 2015

fixed in 74X in cms-sw#7085

@slava77 slava77 closed this as completed Jun 27, 2015
slava77 pushed a commit that referenced this issue Nov 16, 2017
@arizzi
Merge pull request #55 from arizzi/qgl80x
d14734a 
run QGL in 80X
slava77 pushed a commit that referenced this issue Jul 1, 2019
@vkhristenko @fwyzard
constexpr for DetId member functions (#55)
402490c 
Included upstream via cms-sw#23332
slava77 pushed a commit that referenced this issue Mar 9, 2021
@Jingyan95 @skinnari
Add a boolean config param (#55)
eb5915c 
* Enable Maybe Layer

* Setting up the config parameter for maybe layer

* Setting up the config parameter for maybe layer

* Update Settings.h

* Fix Bug

* Maybe layer Update

* Fix Bug

* Remove cout and turn on maybelayer for hybrid

Co-authored-by: Jack Li <jingyan.li@cern.ch>
slava77 pushed a commit that referenced this issue Oct 9, 2021
@cerati
Merge pull request #55 from cerati/cmssw-tests-collisions
f207c20 
cmssw benahcmarking tests
slava77 pushed a commit that referenced this issue Oct 29, 2021
@osschar
Merge pull request #55 from trackreco/updateJSON_forPR365
a5817bf 
Update JSON files for mkFit PR cms-sw#365
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@slava77