-
Notifications
You must be signed in to change notification settings - Fork 20
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
Iss89: Trident and WAB track rates/shapes and track efficiency #179
base: master
Are you sure you want to change the base?
Conversation
…er and histo classes...need to pull in master and separate into separate issues
…nsequence were made
processors/config/baseConfig.py
Outdated
@@ -22,7 +22,7 @@ | |||
help="Which analysis is being run ", metavar="analysis", default="vertex") | |||
parser.add_argument('--infile', '-i', type=str, dest="inFilename", metavar='infiles', nargs="+", | |||
help="Input files, specify on or more.") | |||
|
|||
parser.add_argument('--runnumber', '-r', type=int, dest="runNumber",default=-666) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
option r is used already in various config files, so this will clash when those are used. Can you add this locally to the config files you need it in directly?
…ers hit for tracks and v0s; makes TDirectories for plots split by layers hit; just a checkpoint, not for real use as I will move to the new style, slim tuples next before I push
@bloodyyugo Could you please address @cbravo135 comment so we can move on and close this PR? |
Should be good to go now |
Thank you @bloodyyugo. |
Note: rates & shapes use separate processor (TridentWABAnaProcessor) from track efficiency (TrackEfficiencyProcessor)