-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Adding HZZ electron MVA ID #37429
Closed
Closed
Adding HZZ electron MVA ID #37429
Changes from 12 commits
Commits
Show all changes
347 commits
Select commit
Hold shift + click to select a range
aa640d4
Increase max number of bytes per FED to support full readout mode.
thomreis 44a5ae4
Add new GEM electronics map to Run-3 MC and data
tvami c65bc65
Store variable names in ROOT GBRForest files
Dr15Jones e37cd13
use more the token in accessing collections in SimCalorimetry/HGCalSi…
4070a46
Code check
080f434
Try to fix the discrepancy - thanks to Pedro
f75d13e
Code check
5ac9630
Merge branch 'cms-sw:master' into mvaHZZ_unique
asculac e0a80d7
Adding nonUL indicator
asculac 5011ede
UL miniModifier removed, run2_nanoAOD_preUL added
asculac 26d45cb
One more try
b6ade3a
Removing 106Xv2 campaigns from preUL indicator
asculac 4ed628b
Code check
2f70e44
Applied comments from PR
alexsr-98 47dded9
Merge pull request #37954 from seungjin-yang/GEM-DQM-Eff-Update_from-…
cmsbuild 15b456c
Merge pull request #38043 from Pruthvi-ch/CellUV_partial
cmsbuild 6e0ffa6
Merge pull request #38054 from rovere/FireworksWithWaferDetIds
cmsbuild 39dd061
Merge pull request #38064 from fabiocos/fc-fixvalid
cmsbuild 9e2d377
Fix for Nose
ba9c858
Make a complete scenario for HGCal geometry version V17
37b9747
Applied comments from PR
alexsr-98 aeba46b
Code format
e48b3c4
Merge pull request #38028 from llunerti/from-CMSSW_12_4_0_pre3
cmsbuild 3b007cc
Added esConsumes to RecoMuon/TrackerSeedGenerator classes
Dr15Jones d1245ef
Update ZCounting.cc
ThOliveira23 836184f
Update ZCounting_cff.py
ThOliveira23 241d0c0
Code maintenance for Run III in the DQMOffline ZCounting.
ThOliveira23 fb3ee30
Merge branch 'master' into from-CMSSW_12_5_0_pre1
ThOliveira23 e3c5995
Merge pull request #7 from ThOliveira23/from-CMSSW_12_5_0_pre1
ThOliveira23 78ab7f7
Update ZCounting.cc, code format (l.610 and l.875)
ThOliveira23 928e167
Update ZCounting.cc
ThOliveira23 b1ab480
follow-up to code review: further reduce usage of hardcoded constants
mmusich 11b322b
Remove CMS deprecated warnings from L1Trigger/ConfigUtilities
Dr15Jones fe14891
Merge pull request #38075 from mmusich/fixSiStripHitEfficiencyAtPCL
cmsbuild f6d76b8
Merge pull request #38072 from bsunanda/Phase2-hgx313X
cmsbuild f4084ec
Merge pull request #38086 from jfernan2/124Xbackport38038
cmsbuild c6423b9
Turn ON GEMDigis for GE2/1, turn OFF GEMRecHits for GE2/1
watson-ij 0ca9232
HLT menu development for 12_4_X (branch: 12_5_X)
missirol 30e8864
Add Pythia8 particle gun flat in Pt and Lxy (transverse displacement …
mbluj f559880
Add parameter defining fraction of particles allowed to fly towards d…
mbluj 5dd971d
Fix on issue #38044
quark2 e1801d9
Merge pull request #38055 from Pruthvi-ch/HEsil_correction
cmsbuild ba39140
change merger input as default
466702f
Provide inheriting classes with access to ESConsumesCollector
Dr15Jones c1ff1f2
Fix CMS deprecation warnings in L1TriggerConfig/L1ScalesProducers
Dr15Jones dedad56
Merge pull request #38093 from bsunanda/Phase2-hgx312Y
cmsbuild 6c39638
Compute muon index bits using idx from dataformat
dinyar 83db208
Added # descendents to Error message in TauGenJetProducer
Dr15Jones 9212d0c
Fix CMS deprecation warnings in L1TriggerConfig/DTTrackFinder
Dr15Jones f102b8c
update BeamSpot HLT client to use correct raw data from TCDS
francescobrivio d5788d4
Merge pull request #38010 from srimanob/125_RemoveL1RecoPhase2
cmsbuild 328410a
Merge pull request #38107 from cms-tsg-storm/HLTdevelopment124X_for125X
cmsbuild f2ec19a
putting buffer on the heap
Sam-Harper 1a8c2b7
uGMT DQM changes following removal of ZS
dinyar 76601f2
Hadronic shower input-output comparisons
dinyar 8f6bd04
swapping to more modern conventions
Sam-Harper fe953e7
Merge pull request #38106 from Dr15Jones/moreInfoTauGenJetProducer
cmsbuild 250a6d4
Adapted DQM alerting to new uGMT readout menu
dinyar 30b34a4
Add uGMT muon showers to DQM quality tests
dinyar 785de67
A step toward the V17 version of HGCal Geometry
436f271
Code check
41a9f91
Need to adjust GEM L1T DQM to account for GE2/1 digis
watson-ij ad54d8f
Merge pull request #38080 from mmusich/fixDeltaPhi
cmsbuild 51e566c
Add XML source file describing PPS pixel DAQ mapping for 2022
wpcarvalho 94bd1e5
guard against uint16_t overflows of Pixel ADC(num of electrons) values
missirol 5c1068d
Merge pull request #38035 from dinyar/feature/correct_index_bits_assi…
cmsbuild f834221
Merge pull request #37856 from bsunanda/Run3-sim118E
cmsbuild cea1bb7
Merge pull request #38061 from emiglior/remove_T29_cmssw_12_5_X
cmsbuild 0e72da4
Modifications to allow the Pixel Charge Reweighting at the mixing step
ferencek 0ae02ec
follow-up to code review
mmusich eebada8
change to runDependentForPixel modifier and add the relval 250202.184…
carolinecollard ea77c7a
follow-up to code review
carolinecollard 1198c4a
A safer way following a comment of @fwyzard
quark2 447f938
Add argument to set fraction of particles with sign of z displacement…
mbluj af41170
Merge pull request #38112 from Sam-Harper/DQMFileSaverDBStackIssue_125X
cmsbuild 567f2f5
Merge pull request #38094 from Dr15Jones/deprecatedRecoMuonTrackerSee…
cmsbuild ebc0424
hcal gpu comparison task for online DQM
lwang046 e5e67af
Merge pull request #38088 from thomreis/ecal-gpu-unpacker-buffer-over…
cmsbuild cfaa87f
Merge pull request #37868 from gartung/gartung-mkFit-sse3-vectorize
cmsbuild a7bd0c2
combined commit
civanch 4d980bb
add DoubleMuonLowMass PD to autoAlca
malbouis b07ef28
Merge pull request #38105 from Dr15Jones/deprecatedL1TriggerConfigL1S…
cmsbuild 5c6d4f9
Merge pull request #38109 from Dr15Jones/deprecatedL1TriggerConfigDTT…
cmsbuild 412be6b
Merge pull request #38096 from Dr15Jones/deprecatedL1TriggerConfigUti…
cmsbuild cf0dd45
Merge pull request #38081 from fwyzard/PFClusterEMEnergyCorrector_mis…
cmsbuild 01acc8f
Merge pull request #38076 from alexsr-98/DisplacedMuonSeeding
cmsbuild 4a4ffc1
Correct treatment for partial wafers
f3b3676
Code check
3b080a2
Merge pull request #37987 from civanch/cleanup_geant4_user_code
cmsbuild f0b2f0a
Merge pull request #38063 from cms-tau-pog/CMSSW_12_5_X_tau-pog_rmOut…
cmsbuild 761f6c4
Set missing parameter in the dqmSaver
fwyzard 95400f2
move relval2021 to GRun until V1.2 is ready
Martin-Grunewald 5749b48
Fix for printouts with special (for python) chars
ggovi a3682d8
Merge pull request #38132 from fwyzard/fix_HLTrigger_Configuration_hl…
cmsbuild 76a79d2
Merge pull request #38135 from cms-tsg-storm/AutoHLT125X
cmsbuild 2debcbe
update label of raw TCDS data
francescobrivio b91aadf
Merge pull request #38128 from malbouis/alca-update_autoAlca-125X
cmsbuild d8b1d66
Fixed Geant4 tack information
civanch 36c5752
Split uGMT DQM module into two
dinyar fbd9c63
Improved uGMT DQM plot titles
dinyar d702482
Merge pull request #38113 from missirol/devel_fixZeroADCOfPixelDigi
cmsbuild 72c5200
Merge pull request #38140 from ggovi/conddb-utilities-conddb-tool-fix…
cmsbuild b3c0af9
Move counter initialization and printout+destruction to begin/endJob
makortel ef4bafc
Deal with exception in EventProcessor constructor
Dr15Jones 19466ae
Fix static variable problem in payload inspector code
Dr15Jones 181a48f
Adapting an idea of @fwyzard
quark2 2acfd25
Add new GEM electronics map to all GTs
tvami a60f686
Merge pull request #38150 from wpcarvalho/pps_pixeldaqmapping_2022
cmsbuild 01d80a2
Take care of comments
2c37728
Code check
b453116
Next Step toward V17 version of HGCal Geometry
1b4042a
Changes to get the PU info from OnlineMetaData stream
abhih1 8001666
Code check
1c0bd20
Safeguard for nHits==0
b78c6d5
use cmsonr_lb instead of cms_omds_tunnel for SiStrip O2O unit tests
smuzaffar 0b44f88
Merge pull request #38146 from civanch/fix_g4trachinfo
cmsbuild 1900ac5
RunTheMatrix: fix DeepCore Seeding PU Workflows
mmusich 355678c
Updated Geant4 sensitive calo detector classes for LowEGFlash method
civanch 95fd40a
code format
civanch 80c62ac
Update of L1T menu in TSG GTs to L1Menu_Collisions2022_v1_1_0_xml
missirol b5dfa03
Merge pull request #38162 from smuzaffar/fix-SiStripDAQO2O-test
cmsbuild 6a9931b
Merge pull request #38155 from Dr15Jones/fix_testGetPayloadData
cmsbuild 07bffa5
Bug fixes
grasph 1bd224f
Update GEM packer-unpacker and GEM/DQM for GEMChMap
yeckang 851a170
ran scram build code-format
grasph 154ad5e
Fix memory leak in AlCaHcalHBHEMuonProducer
makortel 5e84d31
minor optimisation
civanch 2b52979
[CORE] [GCC12] Added missing array header
smuzaffar 24f6860
[ALCA-DB] [GCC12] Added missing array header
smuzaffar 0b1eaff
[SIMULATION] [GCC12] Added missing array header
smuzaffar 6b3c6c0
[GEOMETRY-UPGRADE] [GCC12] Added missing array header
smuzaffar 561d771
[DQM] [GCC12] Added missing array header
smuzaffar 2635de4
[DQM-GEOMETRY] [GCC12] Added missing array header
smuzaffar 89b3afc
Avoid reading off end of vector in SimTrackManager
Dr15Jones 4c6f4de
rename ALCARECOStreamPromptCalibProdSiStripHitEfficiency -> ALCARECOS…
mmusich b1f4b2d
Merge pull request #38170 from cms-tsg-storm/updateOfL1TMenuTagInTSGG…
cmsbuild 1a90961
Merge pull request #38144 from Dr15Jones/exceptionInEventProcessorCtr
cmsbuild 4f7813f
Merge pull request #38090 from tvami/GTforGEMChTag
cmsbuild 7d34526
Merge pull request #38184 from smuzaffar/dqm-gcc12-array-inc
cmsbuild bb60e2f
Merge pull request #38157 from bsunanda/Phase2-hgx314A
cmsbuild 1e651ea
Merge pull request #38118 from bsunanda/Phase2-hgx314
cmsbuild 30c9068
Merge pull request #38142 from lwang046/HCAL-GPUOnlineClient
cmsbuild 82cdcb2
Merge pull request #38110 from francescobrivio/alca-collisions_900GeV…
cmsbuild ec10677
Merge pull request #38158 from abhih1/ChangeforPU
cmsbuild bf9c628
Merge pull request #38102 from minerva1993/rpc_dqm_onlineLegacy
cmsbuild ebe5ef4
Revert "[L1T] fix negative shift"
qliphy 430ec5e
Merge pull request #38180 from smuzaffar/core-gcc12-array-inc
cmsbuild 8ac06d8
Merge pull request #37981 from ThOliveira23/master
cmsbuild 9bbee57
Merge pull request #38182 from smuzaffar/simulation-gcc12-array-inc
cmsbuild c6cdcb6
Merge pull request #38183 from smuzaffar/dqm-geometry-gcc12-array-inc
cmsbuild 4561898
Merge pull request #38185 from Dr15Jones/fixSimTrackManager
cmsbuild 6579685
Merge branch 'master' into geometry-upgrade-gcc12-array-inc
smuzaffar 5017de4
Merge pull request #38179 from smuzaffar/alca-db-gcc12-array-inc
cmsbuild 6b83a0f
Merge pull request #38177 from makortel/fixAlCaHcalHBHEMuonProducer
cmsbuild 685e3ee
[CORE] [GCC12] Added missing array header
smuzaffar 91820c9
[SIMULATION] [GCC12] Added missing array header
smuzaffar 5af6ab3
update autoPCL to reflect the new naming scheme
mmusich c011b6e
rename also the Output commands
mmusich afa042e
Merge pull request #38087 from Dr15Jones/deprecateRecoMuonMuonSeedGen…
cmsbuild 13c379c
Merge pull request #38167 from mmusich/fixDeepCorePU
cmsbuild 70a7c4a
Fix static analyzer issues in CalibCalorimetry/EcalCorrelatedNoiseAna…
thomreis d30fdfa
fix bug in tprofile definitions
sroychow 0611eba
Fix ECAL static analyzer issues in CalibCalorimetry/CaloMiscalibTools.
thomreis bfb21a3
Shorten PPS DiamondSampic ALCARECOStream name to fit into DBS databas…
francescobrivio a54d721
Merge pull request #38196 from smuzaffar/core-gcc12-array-inc1
cmsbuild 246ab0f
Fix Sampic DQM channel_per_plane histograms
1d3b5c4
Merge pull request #38186 from mmusich/renameLongAlCaStreamName
cmsbuild 566498e
Merge pull request #38197 from smuzaffar/simulation-gcc12-array-inc1
cmsbuild bc2f90c
protect ALCARECOSiStripCalCosmicsNano for missing tracks (when Tracke…
mmusich 4e30acd
Merge pull request #37823 from CMSTrackerDPG/additionalDigiInfo_from-…
cmsbuild cb55e64
Merge pull request #38209 from francescobrivio/alca-fix_PPSname_PCL
cmsbuild 337dfa1
Merge pull request #38193 from cms-sw/revert-38042-l1t-imath
cmsbuild 1d3400c
Merge pull request #38168 from civanch/updated_lowenergy_gflash
cmsbuild 4334adc
Add good cell definition for v17
Pruthvi-ch 687525d
code checks
Pruthvi-ch d2df24a
Moving constants to HGCalTypes
Pruthvi-ch 993b43e
code checks
Pruthvi-ch a531b25
Merge pull request #38213 from mmusich/protectSiStripCalCosmicsNano_m…
cmsbuild 9341d87
Merge pull request #38115 from dinyar/chore/adapt_ugmt_dqm_for_noZS
cmsbuild c378d9c
Merge pull request #38203 from sroychow/pixheterodqm_profilefix
cmsbuild b54d06d
Merge pull request #37744 from nurfikri89/from1240pre3_nano_jetsrun3_…
cmsbuild 1da1a78
Cleanup redundant edm ParameterSet exist in RecoLuminosity
jeongeun bea6952
[GCC12] Added missing array header
smuzaffar 257472a
Remove HLT out from L1CaloTrigger
cde8d0c
Remove obsolete-unused config
735f85b
Fixed debug printout
civanch 72e637b
avoid overwrite of options PSet from ConfDB in HLT utilities
missirol 73f4e4c
Merge pull request #38159 from AdrianoDee/125_fix_nhits
cmsbuild 4e4f205
Merge pull request #38145 from makortel/CAHitNtupletGeneratorOnGPUdes…
cmsbuild dad56f5
Using char for single characters
quark2 a32da45
Fix static analyzer issues in CalibCalorimetry/EcalLaserSorting.
thomreis 37567ab
Fix static analyzer issue in CondFormats/EcalCorrections.
thomreis 0018731
fix the ROOT type of EventStats monitor element in SiStripGainsPCLWorker
mmusich 1cd337e
run the pixel rundependent modifier before the fast-sim modifications
mmusich be1d3ea
New CondFormat for HG PCL alignment
dmeuser c5bffd3
Merge pull request #38062 from yeckang/GEMRawToDigi_GEMChMap_12_4_X
cmsbuild 2c351f0
Merge pull request #38119 from watson-ij/ge21-digi-off-rec-on
cmsbuild e9fc650
Merge pull request #38181 from smuzaffar/geometry-upgrade-gcc12-array…
cmsbuild 53a138a
Merge pull request #38219 from smuzaffar/gcc12-array1
cmsbuild 6589651
Merge pull request #38091 from Dr15Jones/variablesInRootGBRForestFiles
cmsbuild 2a75dc2
Merge pull request #38234 from mmusich/fixMETypeForEventStats
cmsbuild b1487da
Modify the cfg's in Validation/HGCalValidation to accommodate V17 ver…
c1235c5
Merge pull request #38195 from dmeuser/CondFormats_HG_PCL_PR
cmsbuild 02e7d70
Merge pull request #38225 from missirol/devel_hltDontOverwriteOptions
cmsbuild e90b443
Merge pull request #38236 from mmusich/fixPixelRunDepModifier
cmsbuild 7e79605
add 1D diff plot in vtx compare
sroychow f79ab39
Merge pull request #38217 from Pruthvi-ch/v17_partial_cell
cmsbuild 51e5a9a
Merge pull request #38051 from Dr15Jones/mergedInterpolationFiles
cmsbuild 14f8f61
Merge pull request #38235 from thomreis/ecal-sa-issues-june2022
cmsbuild 70101c3
Update a few cfg's
bffb348
Make use of the goodCell method for partial wafers in defining valid …
03c0e6a
Merge pull request #38224 from civanch/fixed_debug_printout
cmsbuild 091770e
move SuperClusterMerger
swagata87 3b1b1c2
Add Diamond subdet to TotemTimingDetId
66206f3
Returning LepGood_mvaFall17V2noIso
asculac 8e1e5f7
introduce ALCARECOTkAlJetHT and use it in relvals
mmusich acf31a2
Merge pull request #38176 from grasph/dumpraw-bug-fix
cmsbuild 2c45304
Merge pull request #38218 from jeongeun/RecoLumi12X
cmsbuild 0ea3faf
provide new Global Tags for testing new ALCARECOTkAlJetHT workflow
mmusich ea5a8b3
[GCC12] Fix Address of object will never be NULL error
smuzaffar 8cc3294
Rename phase1_2021 to phase1_2022
tvami 330ac5c
Fix CMS deprecation warning in L1Trigger/TextToDigi
Dr15Jones 4a6088c
Merge pull request #38208 from ChrisMisan/from-CMSSW_12_5_0_pre1
cmsbuild 05be9f8
Merge pull request #38263 from smuzaffar/gcc12-address-null-error
cmsbuild ebbd0f2
Merge pull request #38210 from ChrisMisan/dqm_fix_06_2022
cmsbuild 0596072
Merge pull request #38101 from quark2/GEM-onlineDQMRevive37852-12_5_X
cmsbuild 703c881
Merge pull request #38240 from bsunanda/Phase2-hgx315
cmsbuild df67ef7
Merge pull request #38245 from sroychow/heterovtxdqm
cmsbuild c850e19
Merge pull request #38254 from cms-trackeralign/addTkAlJetHT
cmsbuild 8924788
Merge pull request #38267 from Dr15Jones/deprecatedL1TriggerTextToDigi
cmsbuild d3f53ce
Merge pull request #37988 from srimanob/125_RemoveHLTfromL1Tstep
cmsbuild 614996e
condTestRegression: Use correct singularity container for old cmssw r…
smuzaffar 60f2be1
Fix failure of test Alignment/MillePedeAlignmentAlgorithm/test_Create…
iarspider 9704e03
Merge pull request #38026 from swagata87/cleanUpEgamma
cmsbuild f6624b5
Merge pull request #38252 from tvami/RenameTo2022
cmsbuild 34291ba
Merge pull request #38139 from cms-tau-pog/CMSSW_12_5_X_tau-pog_Py8Pt…
cmsbuild 1b6e85e
Merge pull request #37939 from milanchestojanovic/mkFit-PbPb
cmsbuild 812bfa3
Merge pull request #38246 from bsunanda/Phase2-hgx315X
cmsbuild 44ea6b2
Merge pull request #38269 from smuzaffar/fix-condTestRegression
cmsbuild 09a3567
Merge pull request #38271 from iarspider/fix-test_CreateFileLists
cmsbuild 718afff
Adding unique variable for HZZ mva electron ID
asculac fe5e31e
Removed extra modules for run2_miniAOD_80XLegacy
asculac ef782b6
Added default value for mvaHZZIdIso
asculac fba0609
Fixing the HZZ ID problem & adding unique names
asculac 15ece9f
Adding mvaHZZ ID to VID Electron ids
asculac 9b1fdae
Adding nonUL indicator
asculac 4d06bec
UL miniModifier removed, run2_nanoAOD_preUL added
asculac 8b44b8d
Removing 106Xv2 campaigns from preUL indicator
asculac 8f84623
Returning LepGood_mvaFall17V2noIso
asculac 59d227f
Resolve conflict in nano_eras_cff.py
asculac 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 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 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 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
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.
mvaHZZIso = None,
For the first nano-version of the Run3, do you want to have one variable defined with the Run2 training in i.e. that of the 2018 ? later you modify
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.
That was the initial idea, but as @swagata87 pointed out in her reply:
these Run3-specific failures were solved before by simply removing those IDs in Run3, I am not sure if it is possible to have them defined with Run2 training without causing failures. When I try to set it to Run2 value and test it fails with