Skip to content

Commit

Permalink
cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
@civanch
civanch committed Aug 8, 2019
1 parent de79aa6 commit cc23e08
Show file tree
Hide file tree
Showing 4 changed files with 50 additions and 50 deletions.
28 changes: 14 additions & 14 deletions SimG4Core/Application/python/g4SimHits_cfi.py
View file
Expand Up @@ -43,15 +43,15 @@
g4SimHits = cms.EDProducer("OscarMTProducer",
g4GeometryDD4hepSource = cms.bool(False),
NonBeamEvent = cms.bool(False),
G4EventManagerVerbosity = cms.int32(0),
G4EventManagerVerbosity = cms.untracked.int32(0),
UseMagneticField = cms.bool(True),
StoreRndmSeeds = cms.bool(False),
RestoreRndmSeeds = cms.bool(False),
PhysicsTablesDirectory = cms.string('PhysicsTables'),
StorePhysicsTables = cms.bool(False),
RestorePhysicsTables = cms.bool(False),
CheckGeometry = cms.bool(False),
G4CheckOverlap = cms.PSet(
PhysicsTablesDirectory = cms.untracked.string('PhysicsTables'),
StorePhysicsTables = cms.untracked.bool(False),
RestorePhysicsTables = cms.untracked.bool(False),
CheckGeometry = cms.untracked.bool(False),
G4CheckOverlap = cms.untracked.PSet(
OutputBaseName = cms.string('2017'),
MaterialFlag = cms.bool(True),
GeomFlag = cms.bool(True),
Expand All @@ -69,14 +69,14 @@
NodeNames = cms.vstring('World')
),
G4Commands = cms.vstring(),
SteppingVerbosity = cms.int32(0),
StepVerboseThreshold = cms.double(0.1), # in GeV
VerboseEvents = cms.vint32(),
VertexNumber = cms.vint32(),
VerboseTracks = cms.vint32(),
FileNameField = cms.string(''),
FileNameGDML = cms.string(''),
FileNameRegions = cms.string(''),
SteppingVerbosity = cms.untracked.int32(0),
StepVerboseThreshold = cms.untracked.double(0.1), # in GeV
VerboseEvents = cms.untracked.vint32(),
VertexNumber = cms.untracked.vint32(),
VerboseTracks = cms.untracked.vint32(),
FileNameField = cms.untracked.string(''),
FileNameGDML = cms.untracked.string(''),
FileNameRegions = cms.untracked.string(''),
Watchers = cms.VPSet(),
HepMCProductLabel = cms.InputTag("generatorSmeared"),
theLHCTlinkTag = cms.InputTag("LHCTransport"),
Expand Down
35 changes: 18 additions & 17 deletions SimG4Core/Application/src/RunManager.cc
View file
Expand Up @@ -127,10 +127,10 @@ RunManager::RunManager(edm::ParameterSet const& p, edm::ConsumesCollector&& iC)
m_currentRun(nullptr),
m_currentEvent(nullptr),
m_simEvent(nullptr),
m_PhysicsTablesDir(p.getParameter<std::string>("PhysicsTablesDirectory")),
m_StorePhysicsTables(p.getParameter<bool>("StorePhysicsTables")),
m_RestorePhysicsTables(p.getParameter<bool>("RestorePhysicsTables")),
m_EvtMgrVerbosity(p.getParameter<int>("G4EventManagerVerbosity")),
m_PhysicsTablesDir(p.getUntrackedParameter<std::string>("PhysicsTablesDirectory", "")),
m_StorePhysicsTables(p.getUntrackedParameter<bool>("StorePhysicsTables", false)),
m_RestorePhysicsTables(p.getUntrackedParameter<bool>("RestorePhysicsTables", false)),
m_EvtMgrVerbosity(p.getUntrackedParameter<int>("G4EventManagerVerbosity", 0)),
m_pField(p.getParameter<edm::ParameterSet>("MagneticField")),
m_pGenerator(p.getParameter<edm::ParameterSet>("Generator")),
m_pPhysics(p.getParameter<edm::ParameterSet>("Physics")),
Expand All @@ -139,7 +139,7 @@ RunManager::RunManager(edm::ParameterSet const& p, edm::ConsumesCollector&& iC)
m_pStackingAction(p.getParameter<edm::ParameterSet>("StackingAction")),
m_pTrackingAction(p.getParameter<edm::ParameterSet>("TrackingAction")),
m_pSteppingAction(p.getParameter<edm::ParameterSet>("SteppingAction")),
m_g4overlap(p.getParameter<edm::ParameterSet>("G4CheckOverlap")),
m_g4overlap(p.getUntrackedParameter<edm::ParameterSet>("G4CheckOverlap")),
m_G4Commands(p.getParameter<std::vector<std::string> >("G4Commands")),
m_p(p) {
m_UIsession.reset(new CustomUIsession());
Expand All @@ -148,10 +148,10 @@ RunManager::RunManager(edm::ParameterSet const& p, edm::ConsumesCollector&& iC)

m_physicsList.reset(nullptr);

m_check = p.getParameter<bool>("CheckGeometry");
m_WriteFile = p.getParameter<std::string>("FileNameGDML");
m_FieldFile = p.getParameter<std::string>("FileNameField");
m_RegionFile = p.getParameter<std::string>("FileNameRegions");
m_check = p.getUntrackedParameter<bool>("CheckGeometry", false);
m_WriteFile = p.getUntrackedParameter<std::string>("FileNameGDML", "");
m_FieldFile = p.getUntrackedParameter<std::string>("FileNameField", "");
m_RegionFile = p.getUntrackedParameter<std::string>("FileNameRegions", "");

m_userRunAction = nullptr;
m_runInterface = nullptr;
Expand Down Expand Up @@ -193,7 +193,8 @@ void RunManager::initG4(const edm::EventSetup& es) {
bool geoFromDD4hep = m_p.getParameter<bool>("g4GeometryDD4hepSource");
bool cuts = m_pPhysics.getParameter<bool>("CutsPerRegion");
bool protonCut = m_pPhysics.getParameter<bool>("CutsOnProton");
int verb = std::max(m_pPhysics.getUntrackedParameter<int>("Verbosity"), m_p.getParameter<int>("SteppingVerbosity"));
int verb = std::max(m_pPhysics.getUntrackedParameter<int>("Verbosity", 0),
m_p.getUntrackedParameter<int>("SteppingVerbosity", 0));
edm::LogVerbatim("SimG4CoreApplication")
<< "RunManager: start initialising of geometry DD4Hep: " << geoFromDD4hep << "\n"
<< " cutsPerRegion: " << cuts << " cutForProton: " << protonCut << "\n"
Expand Down Expand Up @@ -334,11 +335,11 @@ void RunManager::initG4(const edm::EventSetup& es) {
BeginOfJob aBeginOfJob(&es);
m_registry.beginOfJobSignal_(&aBeginOfJob);

G4int sv = m_p.getParameter<int>("SteppingVerbosity");
G4double elim = m_p.getParameter<double>("StepVerboseThreshold") * CLHEP::GeV;
std::vector<int> ve = m_p.getParameter<std::vector<int> >("VerboseEvents");
std::vector<int> vn = m_p.getParameter<std::vector<int> >("VertexNumber");
std::vector<int> vt = m_p.getParameter<std::vector<int> >("VerboseTracks");
G4int sv = m_p.getUntrackedParameter<int>("SteppingVerbosity", 0);
G4double elim = m_p.getUntrackedParameter<double>("StepVerboseThreshold", 0.1) * CLHEP::GeV;
std::vector<int> ve = m_p.getUntrackedParameter<std::vector<int> >("VerboseEvents");
std::vector<int> vn = m_p.getUntrackedParameter<std::vector<int> >("VertexNumber");
std::vector<int> vt = m_p.getUntrackedParameter<std::vector<int> >("VerboseTracks");

if (sv > 0) {
m_sVerbose.reset(new CMSSteppingVerbose(sv, elim, ve, vn, vt));
Expand All @@ -362,7 +363,7 @@ void RunManager::initG4(const edm::EventSetup& es) {
}

// G4Region dump file name
auto regionFile = m_p.getParameter<std::string>("FileNameRegions");
auto regionFile = m_p.getUntrackedParameter<std::string>("FileNameRegions", "");

// Geometry checks
if (m_check || !regionFile.empty()) {
Expand All @@ -385,7 +386,7 @@ void RunManager::stopG4() {
}
}

void RunManager::produce(edm::Event& inpevt, const edm::EventSetup& es) {
void RunManager::produce(edm::Event& inpevt, const edm::EventSetup&) {
m_currentEvent = generateEvent(inpevt);
m_simEvent = new G4SimEvent;
m_simEvent->hepEvent(m_generator->genEvent());
Expand Down
18 changes: 9 additions & 9 deletions SimG4Core/Application/src/RunManagerMT.cc
View file
Expand Up @@ -61,12 +61,12 @@
RunManagerMT::RunManagerMT(edm::ParameterSet const& p)
: m_managerInitialized(false),
m_runTerminated(false),
m_PhysicsTablesDir(p.getParameter<std::string>("PhysicsTablesDirectory")),
m_StorePhysicsTables(p.getParameter<bool>("StorePhysicsTables")),
m_RestorePhysicsTables(p.getParameter<bool>("RestorePhysicsTables")),
m_PhysicsTablesDir(p.getUntrackedParameter<std::string>("PhysicsTablesDirectory", "")),
m_StorePhysicsTables(p.getUntrackedParameter<bool>("StorePhysicsTables", false)),
m_RestorePhysicsTables(p.getUntrackedParameter<bool>("RestorePhysicsTables", false)),
m_pPhysics(p.getParameter<edm::ParameterSet>("Physics")),
m_pRunAction(p.getParameter<edm::ParameterSet>("RunAction")),
m_g4overlap(p.getParameter<edm::ParameterSet>("G4CheckOverlap")),
m_g4overlap(p.getUntrackedParameter<edm::ParameterSet>("G4CheckOverlap")),
m_G4Commands(p.getParameter<std::vector<std::string> >("G4Commands")),
m_p(p) {
m_currentRun = nullptr;
Expand All @@ -83,7 +83,7 @@ RunManagerMT::RunManagerMT(edm::ParameterSet const& p)
m_stateManager->SetExceptionHandler(new ExceptionHandler());
m_geometryManager->G4GeometryManager::GetInstance();

m_check = p.getParameter<bool>("CheckGeometry");
m_check = p.getUntrackedParameter<bool>("CheckGeometry", false);
}

RunManagerMT::~RunManagerMT() { stopG4(); }
Expand All @@ -98,8 +98,8 @@ void RunManagerMT::initG4(const DDCompactView* pDD,
bool geoFromDD4hep = m_p.getParameter<bool>("g4GeometryDD4hepSource");
bool cuts = m_pPhysics.getParameter<bool>("CutsPerRegion");
bool protonCut = m_pPhysics.getParameter<bool>("CutsOnProton");
int verb =
std::max(m_pPhysics.getUntrackedParameter<int>("Verbosity", 0), m_p.getParameter<int>("SteppingVerbosity"));
int verb = std::max(m_pPhysics.getUntrackedParameter<int>("Verbosity", 0),
m_p.getUntrackedParameter<int>("SteppingVerbosity", 0));
edm::LogVerbatim("SimG4CoreApplication")
<< "RunManagerMT: start initialising of geometry DD4Hep: " << geoFromDD4hep << "\n"
<< " cutsPerRegion: " << cuts << " cutForProton: " << protonCut << "\n"
Expand Down Expand Up @@ -212,7 +212,7 @@ void RunManagerMT::initG4(const DDCompactView* pDD,
initializeUserActions();

// geometry dump
auto writeFile = m_p.getParameter<std::string>("FileNameGDML");
auto writeFile = m_p.getUntrackedParameter<std::string>("FileNameGDML", "");
if (!writeFile.empty()) {
G4GDMLParser gdml;
gdml.SetRegionExport(true);
Expand All @@ -221,7 +221,7 @@ void RunManagerMT::initG4(const DDCompactView* pDD,
}

// G4Region dump file name
auto regionFile = m_p.getParameter<std::string>("FileNameRegions");
auto regionFile = m_p.getUntrackedParameter<std::string>("FileNameRegions", "");

// Geometry checks
if (m_check || !regionFile.empty()) {
Expand Down
19 changes: 9 additions & 10 deletions SimG4Core/Application/src/RunManagerMTWorker.cc
View file
Expand Up @@ -82,8 +82,7 @@ namespace {
void createWatchers(const edm::ParameterSet& iP,
SimActivityRegistry* iReg,
std::vector<std::shared_ptr<SimWatcher> >& oWatchers,
std::vector<std::shared_ptr<SimProducer> >& oProds,
int thisThreadID) {
std::vector<std::shared_ptr<SimProducer> >& oProds) {
if (!iP.exists("Watchers")) {
return;
}
Expand Down Expand Up @@ -152,7 +151,7 @@ RunManagerMTWorker::RunManagerMTWorker(const edm::ParameterSet& iConfig, edm::Co
iC.consumes<edm::LHCTransportLinkContainer>(iConfig.getParameter<edm::InputTag>("theLHCTlinkTag"))),
m_nonBeam(iConfig.getParameter<bool>("NonBeamEvent")),
m_pUseMagneticField(iConfig.getParameter<bool>("UseMagneticField")),
m_EvtMgrVerbosity(iConfig.getParameter<int>("G4EventManagerVerbosity")),
m_EvtMgrVerbosity(iConfig.getUntrackedParameter<int>("G4EventManagerVerbosity", 0)),
m_pField(iConfig.getParameter<edm::ParameterSet>("MagneticField")),
m_pRunAction(iConfig.getParameter<edm::ParameterSet>("RunAction")),
m_pEventAction(iConfig.getParameter<edm::ParameterSet>("EventAction")),
Expand Down Expand Up @@ -236,7 +235,7 @@ void RunManagerMTWorker::initializeTLS() {
}
}
if (m_hasWatchers) {
createWatchers(m_p, m_tls->registry.get(), m_tls->watchers, m_tls->producers, thisID);
createWatchers(m_p, m_tls->registry.get(), m_tls->watchers, m_tls->producers);
}
}

Expand Down Expand Up @@ -299,7 +298,7 @@ void RunManagerMTWorker::initializeThread(RunManagerMT& runManagerMaster, const
tM->SetFieldManager(fieldManager);
fieldBuilder.build(fieldManager, tM->GetPropagatorInField());

std::string fieldFile = m_p.getParameter<std::string>("FileNameField");
std::string fieldFile = m_p.getUntrackedParameter<std::string>("FileNameField", "");
if (!fieldFile.empty()) {
std::call_once(applyOnce, []() { dumpMF = true; });
if (dumpMF) {
Expand Down Expand Up @@ -348,11 +347,11 @@ void RunManagerMTWorker::initializeThread(RunManagerMT& runManagerMaster, const
BeginOfJob aBeginOfJob(&es);
m_tls->registry->beginOfJobSignal_(&aBeginOfJob);

G4int sv = m_p.getParameter<int>("SteppingVerbosity");
G4double elim = m_p.getParameter<double>("StepVerboseThreshold") * CLHEP::GeV;
std::vector<int> ve = m_p.getParameter<std::vector<int> >("VerboseEvents");
std::vector<int> vn = m_p.getParameter<std::vector<int> >("VertexNumber");
std::vector<int> vt = m_p.getParameter<std::vector<int> >("VerboseTracks");
G4int sv = m_p.getUntrackedParameter<int>("SteppingVerbosity", 0);
G4double elim = m_p.getUntrackedParameter<double>("StepVerboseThreshold", 0.1) * CLHEP::GeV;
std::vector<int> ve = m_p.getUntrackedParameter<std::vector<int> >("VerboseEvents");
std::vector<int> vn = m_p.getUntrackedParameter<std::vector<int> >("VertexNumber");
std::vector<int> vt = m_p.getUntrackedParameter<std::vector<int> >("VerboseTracks");

if (sv > 0) {
m_sVerbose.reset(new CMSSteppingVerbose(sv, elim, ve, vn, vt));
Expand Down

0 comments on commit cc23e08

Please sign in to comment.