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.

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

Make default fcls use SCE config #287

Open
marcodeltutto opened this issue Jun 8, 2022 · 2 comments
Open

Make default fcls use SCE config #287

marcodeltutto opened this issue Jun 8, 2022 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed simulation genie or geant4 (g4)

Comments

@marcodeltutto
Copy link
Member

Our "default" fcl files (standard_g4_sbnd.fcl, standard_detsim_sbnd.fcl, etc) have SCE turned off. In the latest production we ended up using "alternative" fcls (g4_sce.fcl, detsim_sce.fcl, etc) that have SCE on.

We should have SCE on in the standard fcls, and use standard fcls as much as possible in production. We should then have alternative configurations without SCE.

@marcodeltutto marcodeltutto added enhancement New feature or request help wanted Extra attention is needed simulation genie or geant4 (g4) labels Jun 8, 2022
@absolution1
Copy link
Contributor

I 100% agree with this and was thinking about this exact thing yesterday.

The X_sce.fcl files enable the SCE services by #include'ing a dedicated FCL file that enables all of the services.

This should not happen when enabling SCE in the standard workflow i.e. the SCE fcl should not be #include'ed in the standard_X_sbnd.fcl files. The reason being that it should not be up to the standard job fcls to define the physics.
Instead, the SCE services need enabling in the upstream service definitions.

@wesketchum
Copy link
Contributor

I agree this is a good idea. Any progress towards this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed simulation genie or geant4 (g4)
Projects
None yet
Development

No branches or pull requests

3 participants