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

[NanoAOD,BTVNano] Btvnano integration as customized module #43401

Closed
wants to merge 674 commits into from

Conversation

Ming-Yan
Copy link
Contributor

PR description:

  • integrate the PFNano to customized NanoAOD module according to the discussion with XPOG
  • additional PFCands, variables to derive SFs are included

time& size report doc

MC (test with /store/mc/Run3Summer22EEMiniAODv4/QCD_PT-15to20_MuEnrichedPt5_TuneCP5_13p6TeV_pythia8/MINIAODSIM/130X_mcRun3_2022_realistic_postEE_v6-v2/2520000/177762d0-23ed-436f-aa0d-a20c33e33dc3.root)
size : 5.61 kb/event
TimeReport ---------- Event Summary ---[sec]----
TimeReport event loop CPU/event = 0.072606
TimeReport event loop Real/event = 0.032194
TimeReport sum Streams Real/event = 0.096403
TimeReport efficiency CPU/Real/thread = 0.563817

data (test with /store/data/Run2022C/BTagMu/MINIAOD/22Sep2023-v1/40000/fc8f31f6-4bf7-4b51-8f6d-ef0833c1e383.root)
size : 5.91 kb/event
TimeReport ---------- Event Summary ---[sec]----
TimeReport event loop CPU/event = 0.101517
TimeReport event loop Real/event = 0.040990
TimeReport sum Streams Real/event = 0.139760
TimeReport efficiency CPU/Real/thread = 0.619164

PR validation:

  • passes the standard runTheMatrix test: runTheMatrix.py -l limited -i all --ibeos

francescobrivio and others added 30 commits June 6, 2023 15:11
…IdFix_avgOutOfLoop

[PileUpJetID] Backport of cms-sw#41854 (Move average calculations outside of jet constituents loop) to 13_0_X
add L1nano NANO in Prompt/Tier0 configuration [130X]
…rk_HV_checks_13_0_X

[13_0_X] Remove stripTrackerHVOn check for BeamSpotLegacy client
…toringPerLS_13_0_X

[13.0.X] add ME-s needed for per-LS certification of Tracking at HLT in `nanoDQMIO` output
…fault

Fixed name of directory in Zmumu.root input file

Use titles from config instead of global tag name for the legend

Produce also PNG images in Zmumu merge
[13.0.X]  Tracker Alignment: all in one Zmumu fixes
…adSafety_130x

[13_0_X] Fix a race condition in StMeasurementDetSet
…e_130x

[13_0_X] Add isAvailable() function to RefToBaseProd
 - add comparison histograms
 - add warning messages, add total FED errors hisogram, add comments on input parametets
 - add 2D map of GPU/CPU error unbalance vs FEDid / FEDerror type
…ion, related to invalid CSC DMB header/trailer data
remove tracking efficiency and fake rate w.r.t. offline plots from list of ME to be outputed by LS [13.0.X]
…artor_13_0_X

[13.0.X] Add `SiPixelRawDataError` monitoring for `gpuValidation` setup
…_dmb_data_corruption_CMSSW_13_0_X

[13_0_X] backport. CSC Unpacker fix for handling of rare CSC data corruption
…2023v1p2p0_130X

Update of L1T pp menu in TSG GTs to `L1Menu_Collisions2023_v1_2_0_xml` [`13_0_X`]
…_Jun23

[13_0_X] Change EMTF Primitive Conversion LUT Assignment for MC
@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-43401/37929

ERROR: Build errors found during clang-tidy run.

@Martin-Grunewald
Copy link
Contributor

-1
wrong branch or rebase needed.

@mmusich
Copy link
Contributor

mmusich commented Nov 29, 2023

@Ming-Yan could you please rebase your commits such that they merge on the master (CMSSW_14_0_X) branch?

@Ming-Yan
Copy link
Contributor Author

@Ming-Yan could you please rebase your commits such that they merge on the master (CMSSW_14_0_X) branch?

HI @mmusich thanks for the comments, I am trying to merge to the master branch but I still have some technical issue where the some files are not present locally. This is still in progress, sorry about the buggy PR

@Ming-Yan
Copy link
Contributor Author

rebase in other branches close this PR

@Ming-Yan Ming-Yan closed this Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment