-
Notifications
You must be signed in to change notification settings - Fork 23
PF hit association #1856
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
bryngemark
wants to merge
44
commits into
trunk
Choose a base branch
from
lk-dev-pfHitAssoc
base: trunk
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.
+443
−130
Open
PF hit association #1856
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
335aef8
Remove comment about unused hit association
bryngemark d3446c4
Add hit, cluster, track indices to pf cand
bryngemark b31e0e3
Add first attempts of pileup hit finding
bryngemark 2477dbc
Fix compilation problems
bryngemark 1d739da
Fix more compilation problems
bryngemark efdae1a
Fix more compilation problems
bryngemark 1148c14
Make idx getters const
bryngemark beb87aa
Add python config
bryngemark 1d30f72
Add missing parameter in python config
bryngemark 682036b
Another missing parameter in python config
bryngemark 780721b
Fix python double vs float nuisance
bryngemark 7361e40
Add output collection to the bus
bryngemark 3e6edf2
Increment class def nb
bryngemark e982cc2
Check that index was set before doing lookup
bryngemark d5f93c6
Add more cluster index setting
bryngemark 21097d5
Improve cluster index setting
bryngemark 26a8f32
Cleanup naming conventions and verbosity
bryngemark dcc6075
Add analysis histograms
bryngemark 7117e56
Add pu finding improvements
bryngemark 7284573
resolve comment merge conflict
bryngemark f0b3c9b
resolve merge conflict
bryngemark 1067ee2
Add first attempts of pileup hit finding
bryngemark 8fb45f9
Fix compilation problems
bryngemark 94b16e5
Fix more compilation problems
bryngemark 3a075b5
Fix more compilation problems
bryngemark 19a2b99
Make idx getters const
bryngemark 00ec9f8
Add python config
bryngemark 4eeb758
Add missing parameter in python config
bryngemark 1a26ed1
Another missing parameter in python config
bryngemark d71dce6
Fix python double vs float nuisance
bryngemark e190e8c
Add output collection to the bus
bryngemark ba37289
Check that index was set before doing lookup
bryngemark 74668fc
resolve merge conflict
bryngemark 1fb3d04
resolve merge conflict
bryngemark 656f7b2
resolve merge conflict
bryngemark 6f0a26e
resolve merge conflict
bryngemark a57c4d2
resolve merge conflict
bryngemark da89764
harmonize variables after merge, now compiles
bryngemark 864fddd
fix float-->int number of bins + double hist def
bryngemark 17e173a
Resolve merge conflict (keep my changes)
bryngemark 608d622
Apply clang-format
github-actions[bot] 9a37140
Update Recon/include/Recon/Event/PFCandidate.h
bryngemark 32fb4db
Update Recon/include/Recon/Event/PFCandidate.h
bryngemark d04e06d
Apply clang-tidy
github-actions[bot] 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
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
Oops, something went wrong.
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.
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.
I suggest to do
a*a
instead ofpow(a,2)
, it's much faster