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
Changed generic 'action' names to be more descriptive. #545
Conversation
…ptive names exception_actions and ExceptionToActionTable.
The following categories have been signed by chrjones (a.k.a. @Dr15Jones on GitHub): Core @cms-git-visualization, @cms-git-core, @cms-git-geometry |
@nclopezo please test. |
Hi, I took this changes on to of CMSSW_7_0_X_2013-08-16-0200, and when I tried to build I got the following error: In file included from /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/PileUp.cc:20:0: /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/SecondaryEventProvider.h:20:14: error: 'ActionTable' has not been declared /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/PileUp.cc: In constructor 'edm::PileUp::PileUp(const edm::ParameterSet&, double, TH1F*, bool)': /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/PileUp.cc:64:92: error: 'ActionTable' was not declared in this scope In file included from /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/SecondaryEventProvider.cc:1:0: /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/SecondaryEventProvider.h:20:14: error: 'ActionTable' has not been declared /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/SecondaryEventProvider.cc:9:22: error: 'ActionTable' has not been declared /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/SecondaryEventProvider.cc: In constructor 'edm::SecondaryEventProvider::SecondaryEventProvider(std::vector&, edm::ProductRegistry&, const int&, boost::shared_ptr)': /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/SecondaryEventProvider.cc:11:86: error: no matching function for call to 'edm::WorkerManager::WorkerManager(boost::shared_ptr, const int&)' /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/SecondaryEventProvider.cc:11:86: note: candidate is: In file included from /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/SecondaryEventProvider.h:4:0, from /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/Mixing/Base/src/SecondaryEventProvider.cc:1: /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/FWCore/Framework/interface/WorkerManager.h:31:5: note: edm::WorkerManager::WorkerManager(boost::shared_ptr, const edm::ExceptionToActionTable&) /build/dmendezl/CMSSW_7_0_X_2013-08-16-0200/src/FWCore/Framework/interface/WorkerManager.h:31:5: note: no known conversion for argument 2 from 'const int' to 'const edm::ExceptionToActionTable&' you can see the complete logs here: |
The recent commit fixes the compilation problem found by David. Please retest. |
Did compile. Bypassing signatures. |
Changed generic 'action' names to be more descriptive.
Add data for CondFormats/JetMETObjects to github repository.
Heppy: adding rl_* control variables (only MT2 related)
Changed the generic name actions:: and ActionTable to the more descriptive names exception_actions and ExceptionToActionTable.