Skip to content

Add NNPuppiTauModel_v1 external package#1

Merged
artlbv merged 1 commit into
cms-hls4ml:mainfrom
artlbv:main
Jun 19, 2026
Merged

Add NNPuppiTauModel_v1 external package#1
artlbv merged 1 commit into
cms-hls4ml:mainfrom
artlbv:main

Conversation

@artlbv

@artlbv artlbv commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

needed by cms-sw/cmsdist#10665

Mirrors the cms-hls4ml externals pattern used by L1TSC4NGJetModel/AXOL1TL: namespaces all hls4ml-generated weights, configs, and the dense/activation call sequence under hls4ml_nnpuppitaumodel_v1, and exposes only the extern "C" create_model()/destroy_model() factory pair across the .so boundary. This avoids ELF symbol clashes (w2/b2/.../w20/b20) when multiple hls4ml models are loaded into the same CMSSW process (cms-sw/cmssw#49632).

NNPuppiTauModel_v1.{h,cpp} are a manual, logic-preserving extraction of TauNNIdHW::EvaluateNN()'s layer call sequence into a standalone namespaced free function, since this model's network body was never kept as a free function in CMSSW (unlike other hls4ml.

Mirrors the cms-hls4ml externals pattern used by L1TSC4NGJetModel/AXOL1TL:
namespaces all hls4ml-generated weights, configs, and the dense/activation
call sequence under hls4ml_nnpuppitaumodel_v1, and exposes only the
extern "C" create_model()/destroy_model() factory pair across the .so
boundary. This avoids ELF symbol clashes (w2/b2/.../w20/b20) when multiple
hls4ml models are loaded into the same CMSSW process (cms-sw/cmssw#49632).

NNPuppiTauModel_v1.{h,cpp} are a manual, logic-preserving extraction of
TauNNIdHW::EvaluateNN()'s layer call sequence into a standalone namespaced
free function, since this model's network body was never kept as a
free function in CMSSW (unlike other hls4ml exports).

Synced from artlbv/NNpuppitau.
@artlbv artlbv merged commit aa4e2da into cms-hls4ml:main Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant