-
Notifications
You must be signed in to change notification settings - Fork 3
feat: apply dRICH and pfRICH quantum efficiency at photon generation (stacking) stage #48
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
Open
Copilot
wants to merge
11
commits into
main
Choose a base branch
from
copilot/enable-drichen-quantum-efficiency
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+101
−39
Open
Changes from all commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
f00a1b6
Enable DRICH generation-stage QE in npsim
Copilot d7fbedc
Address PR feedback on QE interpolation and volume matching
Copilot f795cf6
Require LambdaValues in dd4hep units
Copilot 8eadf60
Normalize RICH efficiency tables to fractions
Copilot 3134a1e
Use std::lerp in stacking interpolation
Copilot 2ed77b2
Remove redundant zero-efficiency branch
Copilot 99c9935
Kill photons outside QE range and promote suppression summary to WARNING
wdconinc 60136b5
Apply QE stacking action to Cherenkov radiator volumes, not sensor vo…
wdconinc 68039cf
Print percentage of suppressed photons in stacking action summary
wdconinc cd0d00a
fix: import DDG4 and use g4units for nm definition
wdconinc 6379d10
fix: from g4units import mm not keV
wdconinc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.