-
Notifications
You must be signed in to change notification settings - Fork 21
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
Introduce WAB Processor #1652
Introduce WAB Processor #1652
Conversation
…tion (i.e. using ECalRecHits + LinearTracking)
…lectron kinematic info if there is only an electron shower)
…M, implemented progress_num to keep track of what reconstruction occurs/why it fails for a given event
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.
Nice work Sanjit!
I have a few comments but nothing major, mostly to make the code more efficient. I didnt comment everywhere but please move to float
s when you can
5659ed8
to
938c0ef
Compare
…ances to x*x; added pass names;cleaned up code in general
938c0ef
to
e418c51
Compare
8016b57
to
1c83de9
Compare
1c83de9
to
90c9ff1
Compare
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.
No further comments, thanks!
I am updating ldmx-sw, here are the details.
What are the issues that this addresses?
This resolves issue #1499