-
Notifications
You must be signed in to change notification settings - Fork 157
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
feat: Extend telescope detector with disc surface #680
Conversation
99eda79
to
2a2724b
Compare
Codecov Report
@@ Coverage Diff @@
## master #680 +/- ##
=======================================
Coverage 49.07% 49.07%
=======================================
Files 331 331
Lines 16568 16568
Branches 7722 7722
=======================================
Hits 8130 8130
Misses 3006 3006
Partials 5432 5432 Continue to review full report at Codecov.
|
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.
Seems fine, I think. I only have a few minor comments.
...etectors/TelescopeDetector/include/ActsExamples/TelescopeDetector/BuildTelescopeDetector.hpp
Outdated
Show resolved
Hide resolved
...etectors/TelescopeDetector/include/ActsExamples/TelescopeDetector/BuildTelescopeDetector.hpp
Outdated
Show resolved
Hide resolved
Examples/Detectors/TelescopeDetector/src/BuildTelescopeDetector.cpp
Outdated
Show resolved
Hide resolved
Hi @paulgessinger , thank you for the suggestions. I have made the update. |
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.
Ok then.
This PR extends the telescope detector to be configurable with either PlaneSurface or DiscSurface. This will allow for reco test with the DiscSurface.
This PR extends the telescope detector to be configurable with either
PlaneSurface
orDiscSurface
. This will allow for reco test with theDiscSurface
.