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

Iss89: Trident and WAB track rates/shapes and track efficiency #179

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

bloodyyugo
Copy link
Contributor

Note: rates & shapes use separate processor (TridentWABAnaProcessor) from track efficiency (TrackEfficiencyProcessor)

@@ -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)
Copy link
Collaborator

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
@pbutti
Copy link
Collaborator

pbutti commented Apr 22, 2024

@bloodyyugo Could you please address @cbravo135 comment so we can move on and close this PR?

@bloodyyugo
Copy link
Contributor Author

Should be good to go now

@pbutti
Copy link
Collaborator

pbutti commented Apr 24, 2024

Thank you @bloodyyugo.
@cbravo135 Could you please check if your comment has been resolved and this is good to go?

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