next-exp / nexus

Geant4 simulation framework of the NEXT Collaboration
5 stars 55 forks source link

Delete BatchSession class #128

Closed gondiaz closed 2 years ago

gondiaz commented 2 years ago

Motivation: trying to compile NEXUS with Geant4 11, I found a first compilation error in BatchSession.cc, since G4String.cc class was modified in this new G4 version.

Since BatchSession is a difficult module, I propose to delete it and redefine NexusApp::ExecuteMacroFile as shown.

jmalbos commented 2 years ago

We added that class to modify the default behavior of Geant4, stopping the initialization if an unknown parameter is found in a macro file. I'll have a look at the error you mention to see if there's an easy way to fix it rather than eliminating the class altogether.

gondiaz commented 2 years ago

Ok, I think it can be modified instead, I close this PR.