Skip to content

Conversation

@liuxiaoyuyuyu
Copy link

PR description:

This PR adds a vertex smearing for 2025 OO MC. Beamspot position derived from runs 394153-394217, excluding runs with VdM scans, i.e 394154 and 394217. BPIX barycentre locations are officially provided by Tracker Alignment Group (twiki)

PR validation:

This PR has been modeled after a similar PR for 2025 pO collisions: #48480

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:

We will need a backport for CMSSW_15_0_X

@cmsbuild
Copy link
Contributor

cms-bot internal usage

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @liuxiaoyuyuyu for master.

It involves the following packages:

  • Configuration/StandardSequences (operations)
  • IOMC/EventVertexGenerators (simulation)

@antoniovilela, @civanch, @cmsbuild, @davidlange6, @fabiocos, @kpedro88, @mandrenguyen, @mdhildreth, @rappoccio can you please review it and eventually sign? Thanks.
@GiacomoSguazzoni, @Martin-Grunewald, @VinInn, @VourMa, @dgulhan, @fabiocos, @felicepantaleo, @grzanka, @makortel, @missirol, @mmusich, @mtosi, @rovere, @sameasy, @slomeo this is something you requested to watch as well.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@@ -1,7 +1,7 @@
import FWCore.ParameterSet.Config as cms
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think a new file should be created rather than editing the 2023PbPb file to contain 2025OO. In fact, the above modification to Configuration/StandardSequences/python/VtxSmeared.py will not run properly unless this is done.

@liuxiaoyuyuyu
Copy link
Author

The vertex smearing parameters need to be updated again after the BS fit. So I am closing this PR, will open a new one after the final values are obtained.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants