Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
    • Locked files
  • Jira
    • Jira
  • Merge requests 138
    • Merge requests 138
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • atlasatlas
  • athenaathena
  • Merge requests
  • !20300

Small bugfix in TrackJets overlap removal of SUSYTools

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Johannes Junggeburth requested to merge SUSYTools_dR into 21.2 Jan 11, 2019
  • Overview 13
  • Commits 2
  • Pipelines 2
  • Changes 1

Hi @ykeisuke, @oabouzei,

happy new year first of all. During a chat with @pgadow, I noticed that the new feature of the track jet overlap removal has a small bug in it, since the dR is calcutaed via

dR^{2} = (eta1 - eta2)^{2} + (phi1 -phi2)^{2};

if jet1 one has phi of pi + depsilon and the oher -phi-depsilon, then the equation above gives roughly 2pi. I also switched from using the auto loop to the iterators directly since this allows us to avoid pairing the jets twice.

I include @schae, @jgonski to this request to have a final pairs of eyes look at this small fix.

Cheers,

Johannes

Edited Jan 11, 2019 by Johannes Junggeburth
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: SUSYTools_dR