Skip to content

Commit

Permalink
BUGFIX: PIDLookup: fix proton sampling threshold (#1728)
Browse files Browse the repository at this point in the history
Electron threshold was used for sampling protons due to what looks like
some copy-paste error. The threshold reported in ParticleID for 2212 is
correct, however. Basically, this is only issue for
`ReconstructedParticles.PDG`.

### Briefly, what does this PR introduce?


### What kind of change does this PR introduce?
- [x] Bug fix (issue #__)
- [ ] New feature (issue #__)
- [ ] Documentation update
- [ ] Other: __

### Please check if this PR fulfills the following:
- [ ] Tests for the changes have been added
- [ ] Documentation has been added / updated
- [ ] Changes have been communicated to collaborators

### Does this PR introduce breaking changes? What changes might users
need to make to their code?
No

### Does this PR change default behavior?
Yes
  • Loading branch information
veprbl authored Feb 7, 2025
1 parent 04551bd commit 9a54a66
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/algorithms/pid_lut/PIDLookup.cc
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ void PIDLookup::process(const Input& input, const Output& output) const {
identified_pdg = 321; // kaon
recopart.setParticleIDUsed((*partids_out)[partids_out->size() - 2]);
} else if (random_unit_interval < (entry->prob_electron + entry->prob_pion +
entry->prob_kaon + entry->prob_electron)) {
entry->prob_kaon + entry->prob_proton)) {
identified_pdg = 2212; // proton
recopart.setParticleIDUsed((*partids_out)[partids_out->size() - 1]);
}
Expand Down

0 comments on commit 9a54a66

Please sign in to comment.