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

Vertex smearing for PbPb MC based on 2023 PbPb data #43211

Closed
wants to merge 260 commits into from

Conversation

claytoniousfunk
Copy link
Contributor

PR description:

  • This PR adds a vertex smearing for 2023 PbPb MC.
  • Beamspot position derived from run 374719, barycentre location provided by Tracker Alignment Group (cmsTalk)

PR validation:

  • This PR has been modeled after a similar PR from 2022: link

@boundino @mandrenguyen

wddgit and others added 30 commits July 26, 2023 17:50
…sourcing_13_2_X

[13_2_X] Improve discover method in cond2xml for read-only and patch releases
[13_2_x] PPS suppress LogErrors in Pixel unpacker
…s_in_alignment_validation

[13.2.X] Support HLT tracks in alignment validation
…Payloads_fix_BACKPORT_13_2

[13_2_X] fixed comparePayloads in LHInfo* PopCons
…erMeas_132X

add layer-TSOS compatibility check in `LayerMeasurements` [`13_2_X`]
…e_v2

[13_2_X] Add HARVEST step to SKIM WF for Run3
…tag_in_2023MC_gts

[13_2_X] Update 2023 MC GTs with the fixed L1T tag and re-snapshot the data GTs
…AreaFinder_13_2_X

[13.2.X] clear `badPixelDetsBarrel_` and `badPixelDetsEndcap_` in `PixelInactiveAreaFinder`
…rity-checks-part2

ECAL skip GPU unpacking of the rest of the block if a bad block is detected - 132x
…_post41632_132X

fix configuration of GPU Pixel unpacker in Run-3 HLT menus post-cms-sw#41632 [`13_2_X`]
…h_static_assert

Replace the SFINAE check with static_assert [13.2.x]
…132x

[Backport 13_2_X] Fix emission veto hook
Fix error code on fallback file open error with test, 13_2_X backport
[13_2_X] Removal of StringCutObjectSelector from Muon trigger DQM
The ESProducer siPixelQualityESProducer can be removed from
the cff fragments of the HLT configurations, as it is provided by the cff file

Configuration.StandardSequences.FrontierConditions_GlobalTag_cff

This avoids picking up the version of siPixelQualityESProducer defined in ConfDB
when a command like the following is used.

cmsDriver.py step2 --conditions 130X_mcRun3_2023_realistic_relvals2023D_v1 \
--datatier GEN-SIM-DIGI-RAW --era Run3_2023 --eventcontent FEVTDEBUGHLT \
--filein "dbs:/RelValTTbar_14TeV/CMSSW_13_0_11-130X_mcRun3_2023_realistic_relvals2023D_v1_RV204-v1/GEN-SIM" \
--fileout "file:step2.root" --geometry DB:Extended --nThreads 8 --no_exec --number 1 \
--pileup 2023_Fills_8807_8901_ProjectedPileup_PoissonOOTPU \
--pileup_input das:/RelValMinBias_14TeV/CMSSW_13_0_11-130X_mcRun3_2023_realistic_relvals2023D_v1_RV204-v1/GEN-SIM \
--python_filename step_2_cfg.py --step DIGI:pdigi_valid,L1,DIGI2RAW,HLT:run3:/dev/CMSSW_13_0_0/GRun/V151

Using HLT:[db]:[config] (as opposed to HLT:[menuKeyword], e.g. HLT:GRun)
leads to a call to the function loadHltConfiguration
after the process has loaded FrontierConditions_GlobalTag_cff,
thus overwriting the ESProducer from the latter cff with
the one defined in the HLT configuration as taken from ConfDB.
For details on a concrete use case, see

https://its.cern.ch/jira/browse/PDMVRELVALS-204
Update of the scripts

HLTrigger/Configuration/test/getHLT.sh , and
HLTrigger/Configuration/test/getFrozenHLT.sh

to use an associative array to specify which
HLT configurations should be downloaded from ConfDB.

This allows for more flexibility for non-standard use cases
(e.g. having to specify different version numbers for different subtables).
In getFrozenHLT.sh , this change also avoids having
to specify the list of configurations in a separate string
(which was named TABLES prior to this commit):
now, it is sufficient to add a new configuration to the associative array.

The downside of using associative arrays is that
the configurations are not downloaded anymore in a specific order,
but said order is (and should be) unimportant in this context.
mmusich and others added 5 commits October 7, 2023 13:28
…OV_misalignment_fix_and_duringFill_fix

[13_2_X] Fix LHCInfoPerLS PopCon incorrectly matching LS between OMS and PPS db and fix LHCInfoPer* PopCons not recoginizing ongoing fills in duringFill mode
…as_forHI

[13.2.X] Use `TriggerResultsFilterFromDB`-based selection for Heavy AlCaRecos in `pp_on_AA`
[13_2_X]Remove rebin from SiStripBadAPVandHotStripAlgorithm
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment