forked from cms-sw/cmssw
-
Notifications
You must be signed in to change notification settings - Fork 1
Integration of e/g CompositeID in 12.5.X #110
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
Merged
gpetruc
merged 70 commits into
p2l1pfp:L1PF_12_5_X
from
cerminar:composite_eleid_125X_v0
Apr 4, 2023
Merged
Changes from 37 commits
Commits
Show all changes
70 commits
Select commit
Hold shift + click to select a range
c848045
starting point composite ID emulator
171cc91
cast floats to apfixed, run inference and compare output
ef982cf
add scheleton to plug in CompositeID
cerminar 7edb2b7
incorporate compositeID as alternative to tkElectron, using skeleton …
4e7c57f
fix DR and add some verbosity
cerminar 2e89963
equip to write decoded tracks to the event
cerminar 55c4544
add calo coordinates to PFCandidates
cerminar 5d19697
trying to move normalizations to different function
62d8ade
implement two BDT WPs
3ac8427
fixes
0d4e16d
Mapping of scores and WPs for consistent comparisons
108deea
trivialize the CompositeCandidate interface
cerminar e89f8fc
remove the cut on the CompositeID BDT and store the score in the TkEl…
cerminar 98932ee
Remove pt10 cut from composite candidates
525233e
remove duplicated files
cerminar f8e4cea
remove duplicated files
cerminar 42d7ce2
fix typo
cerminar 32aa4c7
add missing headers
cerminar bbc27da
first implementation of the new data-formats for CompositeID input fe…
cerminar 0b28796
reorder the code for adding the cluster-shape variables used by the c…
cerminar cd9bd45
temporary changes to run emulator from firmware testbench
cerminar 0934ff3
make # of composite candidates per cluster configurabel
885ca19
add parameter for # of composite candidates per cluster
cerminar 56ed6f7
change to new model with max tree depth of 4
8df4f6a
add variales for composite ID to HGCAL raw dataformat
c76cda6
Replace egamma BDT with model using no input scaling
b5ee230
Update egamma BDT to model trained on hw values. Use inversion fn fro…
47d4350
fix typo
cerminar 5874bb5
switch to rounding for consisntency with HGC raw data format
cerminar c5b620f
fix assignment of cluster variables
cerminar 1549f9f
add chi2 variables from track word, address some rounding issues in t…
cerminar 3ed4d4f
mostly cosmetics
cerminar 42dbbf2
move the chi2 back to be an emulator only quantity
cerminar 0a2e3e9
fix constexpress
a45417a
code-format and remove unused parameters from CombinedID configuration
cerminar 9b6c9f9
back to 1000 events
cerminar 3096e6a
code-format
cerminar c9102cd
Drop includes not needed any more
cerminar 4eba6d6
Event version is back to 12
cerminar a72fe0c
Introduce typedef for bdt features type + some std here and there
cerminar fad8b55
code format
cerminar dc0c262
undo changes to HGC ntuplizer
cerminar ccbf885
add slim version of packed Tk and Calo objs
cerminar aea1f6a
protect conversion for empy (0) words
cerminar 466463b
Avoid code duplication or SLIM objects and other cosmetics
cerminar ac2694c
make unpacking of the addtional track variables configurable for barr…
cerminar d6a0d7e
Reinstate conifer runtime exception. Use cms::Exception in CMSSW cont…
thesps b84d29b
Merge pull request #13 from thesps/composite_eleid_125X_v0+cnf-fix
cerminar c7b819c
code format
cerminar e75fcf2
Handle BDT WPs and do some cosmetics
cerminar ac9470b
code format and fix python
cerminar c5e1296
cosmetics
cerminar 8dfef6d
new model with srrtot rescaled, binned chi2 variables and without z0
c2ba610
Merge pull request #14 from Pmeiring/composite_eleid_125X_v0_newmodel
cerminar 7cb2bd4
fix srr conversion back to float
cerminar b9e3ae9
Fix constexpr
cerminar feb0b83
code-format
cerminar 4f0231c
Set working points. Don't transform WPs or BDT score
thesps 64e9f43
Merge pull request #15 from thesps/composite_eleid_125X_v0_wps
cerminar fb34575
attempt to rounding problems in scaling srrtot
cerminar b3dde57
workaround to propagate BDT score to Layer2 objects
cerminar f64aac1
remove uneeded header
cerminar 53081b5
make names of score variables more future proof
cerminar ce0ce6e
obsolete comment
cerminar 73b2049
Fix type used in 1/pT
thesps 8eaa9df
Merge pull request #16 from thesps/composite_eleid_125X_v0_wps
cerminar 030489a
update the model: now built for using the HW input values
cerminar 846567b
update tight WP
cerminar 8d77d91
Update tight WP for 90%eff
cerminar f2aa770
Duplicate HGCAl TkeElectrons to run legacy Elliptic Algorithms for Ph…
cerminar 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
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
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.
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.
12?