#61 closed Bug (fixed)
Isolation routine does not compute closest track correctly
| Reported by: | Sourav Mandal | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | Delphes code | Version: | |
| Keywords: | Cc: |
Description
We at IPMU found another bug in the isolation routine (RESOLution::Isolation in src/SmearUtil.cc:1429). While the previous bug (Ticket #46) skipped tracks inside the isolation cone, giving a wrong result for ptiso, there is another bug that causes it to not find the closes tracking, giving the wrong result for the isolation flag.
On line 1440, the saved smallest DeltaR ("deltar") is being overwritten by the current DeltaR ("genDeltaR") on *every track*. This should only occur for the current track if genDeltaR < deltar.
A patch against Delphes 2.0.2 is attached.
PS: I mistakenly attached this patch to Ticket #46; please delete it.
Attachments (1)
Change History (4)
by , 12 years ago
| Attachment: | delphes-isolation.patch added |
|---|
comment:1 by , 12 years ago
follow-up: 3 comment:2 by , 12 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Hi,
Your patched has been merged and will be included in the next release.
Thanks !
comment:3 by , 12 years ago
Replying to favereau: play casino
Hi,
Your patched has been merged and will be included in the next release.
Thanks !
Great thanks.
"closes tracking" -> "closest track"