-
Notifications
You must be signed in to change notification settings - Fork 2
Revised TOPO model for hh->bbWW->bbmunuqq #2
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
base: main
Are you sure you want to change the base?
Conversation
TOPO_HHbbWW_1mu_v3/Makefile
Outdated
| EMULATOR_EXTRAS := /cvmfs/cms.cern.ch/el8_amd64_gcc11/external/hls4mlEmulatorExtras/1.1.1-6933fcc7cdb4cdd5a649bd6579151d1b/ | ||
| AP_TYPES := $(EMULATOR_EXTRAS)/include/ap_types | ||
| HLS_ROOT := /cvmfs/cms.cern.ch/el8_amd64_gcc11/external/hls/2019.08-fd724004387c2a6770dc3517446d30d9 |
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 don't think it's okay to use hard-coded paths to /cvmfs (otherwise, how does one build this package on nodes without /cvmfs access, and/or using the latest version of the hls or hls4mlEmulatorExtras externals ?).
For comparison, see here what is done for AXOL1TL.
If so, this should be fixed everywhere in this package (and the fix should be propagated to all the relevant cmsdist branches, i.e. 16_0_X and 16_1_X as of now).
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.
hi Marino, thank you for looking into this!
I made the changes here: LukasEbeling#2
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.
Thanks, @artlbv.
Removed INCLUDES variable from Makefile
Fix makefile to not use explicit hls4ml/hls paths
Adding model v3


This PR adds a revised model (versions 2 and 3) including the following changes:
hwQualof muons added as featuresetaAtVtxandphiAtVtxare used instead ofetaandphifor muonsphicoordinate of HT removedFor model v3 the architecture was revised to meet the stringent latency requirements. Instead of three hidden layers, the model included two layers with 128 + 64 nodes.
The model was trained on 2025 MC samples: