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

implemented additional options to command line parameters #29

Merged
merged 2 commits into from
Mar 7, 2022

Conversation

lobis
Copy link
Member

@lobis lobis commented Mar 5, 2022

lobis 20

Implemented two optional command line parameter options for restG4:

-o output.root -> to select the output file, overwritting the rml
-g geometry.gdml -> to select the geometry file, overwritting the rml

These options can be useful for example in interactive environments such as a notebook where you would like to change these options for different cells without changing the rml.

@lobis lobis requested a review from jgalan March 5, 2022 17:23
@jgalan jgalan requested a review from nkx111 March 5, 2022 17:56
@jgalan
Copy link
Member

jgalan commented Mar 5, 2022

Perhaps it would be interesting to introduce a generic way to be able to control any parameter through command line.

Doing something like:

restG4 --g4metadata:gdml_file=geometry.gdml --run:outputFileName=output.root

If we achieve something generic, then we could have full freedom. I.e. I could fix also the number of events as follows:

restG4 --g4metadata:Nevents=100

@lobis
Copy link
Member Author

lobis commented Mar 7, 2022

Perhaps it would be interesting to introduce a generic way to be able to control any parameter through command line.

Doing something like:

restG4 --g4metadata:gdml_file=geometry.gdml --run:outputFileName=output.root

If we achieve something generic, then we could have full freedom. I.e. I could fix also the number of events as follows:

restG4 --g4metadata:Nevents=100

I think this is a great idea.

@lobis lobis merged commit ac3fcb3 into master Mar 7, 2022
@lobis lobis deleted the lobis-command-line-params branch March 7, 2022 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants