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

Realistic vertex smearing for 2015 PbPb MC (81X) #15224

Merged

Conversation

mandrenguyen
Copy link
Contributor

Update the vertex smearing based on final beamspot fits to the data (taking into account the shift of the pixel detector). The corresponding reco beamspot will be put in a GT and added shortly.
Since the reco beamspot tag is not yet in place, the HI reco relval workflow is not updated to use this vertex smearing. I tested the PR with:

cmsDriver.py MinBias_8TeV_pythia8_TuneCUETP8M1_cff --conditions auto:run2_mc_hi --scenario HeavyIons -n 1 --eventcontent RAWSIM --relval 2000,1 -s GEN,SIM --datatier GEN-SIM --beamspot RealisticHICollision2015 --customise SLHCUpgradeSimulations/Configuration/postLS1Customs.customisePostLS1_HI

There will also be a backport to 75X.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mandrenguyen for CMSSW_8_1_X.

It involves the following packages:

Configuration/StandardSequences
IOMC/EventVertexGenerators

@cmsbuild, @civanch, @franzoni, @mdhildreth, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @Martin-Grunewald, @dgulhan this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@civanch
Copy link
Contributor

civanch commented Jul 19, 2016

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 19, 2016

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/14160/console

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@civanch
Copy link
Contributor

civanch commented Jul 19, 2016

+1

@civanch
Copy link
Contributor

civanch commented Jul 19, 2016

@mandrenguyen , you removed VtxSmearedUpdatedHICollision2015_cfi, please, confirm that this "Updated" beam-spot is not used anywhere.

@mandrenguyen
Copy link
Contributor Author

@civanch : Indeed it was never used anywhere. We intended to have an improved guess, but in the end decided to just use the initial predictions. Now that the data is recorded, I think "Realistic" is the appropriate name.

@davidlange6 davidlange6 merged commit 6e413a4 into cms-sw:CMSSW_8_1_X Jul 20, 2016
@mandrenguyen mandrenguyen deleted the realisticVtxSmearingPbPb2015_81X branch March 4, 2022 10:12
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.

None yet

4 participants