-
Notifications
You must be signed in to change notification settings - Fork 29
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
spill simulation #111
spill simulation #111
Conversation
@krwood I think keeping track of two primary simulation scripts is going to be kind of a nightmare. Since the differences are fairly limited (see below), it seems like it might make more sense to try and work these into configuration parameters. I think the end goal for simulate_pixels.py should be to be an entry point into larndsim, and have most of the simulation settings configurable within the yaml files that it consumes. There's also probably a clever way to create command line arguments that take precedence over the yaml file (handy, but is probably a low priority for the moment). I think you could do something like the following:
There are also lots of configuration values (e.g. all the gain/vref/vcm/etc in fee.py) that should also be made configurable from the yaml file, but that might be less urgent.
The event separator stuff should all work fine though, but I might complain about the changing capitalization scheme... |
I completely agree with your organizational comments. I was planning on merging the two |
Ah ok. Either is fine by me. |
@drinkingkazu would you mind giving this PR a look over and letting me know if it looks like it will work for your purposes? Your outstanding PR will need to be revised if you're happy to merge this branch in. Instead of making the EVENT_SEPARATOR argument configurable from the command line, this PR reads it in from a configuration file. |
Adds new simulation script and functionality for simulating spills from neutrino beams