DUNE / larnd-sim

Simulation framework for a pixelated Liquid Argon TPC

Apache License 2.0

10 stars 29 forks source link

support eventID separator as a command-line option #109

Closed drinkingkazu closed 1 year ago

drinkingkazu commented 1 year ago

a minor change so that a user doesn't have to locally install larnd-sim nor modify the system path install to use a string value "eventID" as a separator (the default kept as spillID)

drinkingkazu commented 1 year ago

Added another piece of change that logs the time and memory usage over simulation steps for understanding the performance in laymans term w/o using nvprof outside.

krwood commented 1 year ago

111 reads in `EVENT_SEPARATOR` and other formerly hard coded configurations through a configuration file instead of from the command line. I would like to merge that in and then revise this PR to get your memory logger in. How does that sound to you?

drinkingkazu commented 1 year ago

@krwood config file sounds good. yes please merge the mem logger! sorry i was traveling till April...

drinkingkazu commented 1 year ago

Closing to open a new PR for memory logger

DUNE / larnd-sim

support eventID separator as a command-line option #109

111 reads in EVENT_SEPARATOR and other formerly hard coded configurations through a configuration file instead of from the command line. I would like to merge that in and then revise this PR to get your memory logger in. How does that sound to you?

111 reads in `EVENT_SEPARATOR` and other formerly hard coded configurations through a configuration file instead of from the command line. I would like to merge that in and then revise this PR to get your memory logger in. How does that sound to you?