search

next-exp / nexus

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

Possibility of not specifying the persistency manager #194

Closed paolafer closed 1 year ago

paolafer commented 1 year ago

This PR removes the need of specifying a persistency manager to run a simulation. It can be useful for debugging, an example of which has been added in a pair of macros. Additionally, the names of two configuration parameters of the PersistencyManager class have been changed to follow the current convention.

gonzaponte commented 1 year ago

@amirbh61 also needs this. Can I help push it forward?

soleti commented 1 year ago

@gonzaponte thanks for the reminder, I'll review this by Monday, but feel free to add yourself as a second reviewer if you want.

paolafer commented 1 year ago

Looks good to me (although I have limited experience with nexus)

If you're happy with the PR, please, approve it (Review changes --> Approve, in the Files changed tab).