Skip to content
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

TEST #3

Draft
wants to merge 1,640 commits into
base: CMSSW_13_3_X
Choose a base branch
from
Draft

TEST #3

wants to merge 1,640 commits into from

Conversation

smorovic
Copy link
Owner

PR description:

PR validation:

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:

Before submitting your pull requests, make sure you followed this checklist:

mmusich and others added 30 commits February 16, 2024 12:11
…ringAtHLT_14_0_0_pre3

[14.0.X] Add customize to include Alpaka HCal PF Clustering at HLT
… it to be used outside of ALPAKA_ACCELERATOR_NAMESPACE

I.e. repeat cms-sw#43310 on PortableMultiCollection.
Move the ROOT read rules for the ECAL digis SoA host collection from
classes_def.xml to classes.cc.
Move the ROOT read rules for the ECAL uncalibrated rechits SoA host collection
from classes_def.xml to classes.cc.
Move the ROOT read rules for the PF rechits SoA host collections from
classes_serial_def.xml to classes_serial.cc.
mmusich and others added 16 commits March 15, 2024 11:39
Implement AlpakaBackendProducer, an empty alpaka-based EDProducer whose
only purpose is to save in the event what alpaka backend has been used.

Implement AlpakaBackendFilter, an EDFilter that selects events based on
the alpaka backend used to run a previous producer.

Implement a unit test for both modules.
Make repack process be able to generate new data tiers: L1SCOUT, HLTSCOUT
Port from CUDA to Alpaka the changes in cms-sw#42428 by Silvio Donato.
Delete legacy modules and tasks after defining the alpaka ones.
Clean up white space and indentation.
Customise the Status_On{CPU|GPU} HLT paths and the
DQM_{Ecal|Hcal|Pixel}Reconstruction HLT paths to use the AlpakaBackendProducer
and AlpakaBackendFilter modules.
This name is unique, the previous name clashed with AlpakaTest/plugins/BuildFile.xml

Co-authored-by: Matti Kortelainen <[email protected]>
- set a default value in python
- have the cfi generator still add the default blank string
@smorovic smorovic force-pushed the 14_0_X-hlt_diphoton_mva branch from 71f0046 to 5ff2a37 Compare March 19, 2024 12:47
valsdav and others added 10 commits March 19, 2024 14:44
…omSoAAlpaka_along_42428_140x

Fix track sort order in PixelTrackProducerFromSoAAlpaka [14.0.x]
[14_0_X] Add NanoAOD flavours parameter to PromptReco
…ilter_14_0_x

Implement AlpakaBackendProducer and AlpakaBackendFilter [14.0.x]
…paka_140x

Update the HLT alpaka customisation [14.0.x]
[14_0_X] Produce r9 and trackIso(hollow cone) in EGamma development path
[backport] DeepTau - Do not call TF inference with empty grid
Add ESInputTag to VPSet default added to cfi file [14_0]
* Photon XGBoost MVA estimator and producer.
Implemented using native XGBoost C API for photon MVA score
Currently diphoton score filter is not yet submitted.

Model files will be added to the cms-data

XGBoost Double photon combination filter

fix module name

code-format

fix comments from maintainers

Update HLTrigger/Egamma/plugins/HLTEgammaDoubleXGBoostCombFilter.cc

Co-authored-by: Marco Musich <[email protected]>

Update HLTrigger/Egamma/plugins/HLTEgammaDoubleXGBoostCombFilter.cc

Co-authored-by: Marco Musich <[email protected]>

Update HLTrigger/Egamma/plugins/HLTEgammaDoubleXGBoostCombFilter.cc

Co-authored-by: Marco Musich <[email protected]>

Update HLTrigger/Egamma/plugins/HLTEgammaDoubleXGBoostCombFilter.cc

Co-authored-by: Marco Musich <[email protected]>

Update HLTrigger/Egamma/plugins/HLTEgammaDoubleXGBoostCombFilter.cc

Co-authored-by: Marco Musich <[email protected]>

Update HLTrigger/Egamma/plugins/HLTEgammaDoubleXGBoostCombFilter.cc

Co-authored-by: Marco Musich <[email protected]>

Update RecoEgamma/PhotonIdentification/plugins/PhotonXGBoostProducer.cc

Co-authored-by: Marco Musich <[email protected]>

Update RecoEgamma/PhotonIdentification/plugins/PhotonXGBoostProducer.cc

Co-authored-by: Marco Musich <[email protected]>

Update RecoEgamma/PhotonIdentification/plugins/PhotonXGBoostProducer.cc

Co-authored-by: Marco Musich <[email protected]>

Update RecoEgamma/PhotonIdentification/plugins/PhotonXGBoostProducer.cc

Co-authored-by: Marco Musich <[email protected]>

Update RecoEgamma/PhotonIdentification/plugins/PhotonXGBoostProducer.cc

Co-authored-by: Marco Musich <[email protected]>

tidy up and sort headers

code-format

Update RecoEgamma/PhotonIdentification/plugins/PhotonXGBoostProducer.cc

Co-authored-by: Marco Musich <[email protected]>
XGBoost Photon MVA unit test

Unit test for v1 model files in cms-data
Change of ntree limit parameter
@smorovic smorovic force-pushed the 14_0_X-hlt_diphoton_mva branch from bce6bae to 46e56ea Compare March 21, 2024 12:02
smorovic pushed a commit that referenced this pull request May 8, 2024
* remove comments, set constants constant

* remove CASTOR cleaner and merger

* Remove single generator customize function. It is not used anymore since the generator step is splitted in 3 steps.

* remove unneeded filters

* optimize embedding correction calculation

* optimize selector loops

* format cpp code with scram build code-format

---------

Co-authored-by: cwinter <[email protected]>
smorovic pushed a commit that referenced this pull request Jul 15, 2024
Update prompt and displaced muon names to reflect GMT changes
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.