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

Consumes migration and GenHIEventProducer forward port from 75X #14719

Merged
merged 2 commits into from Jun 24, 2016

Conversation

mandrenguyen
Copy link
Contributor

This package seems to have escaped consumes migration, presumably as these filters weren't being used.
Two of these filters are now needed again:
BetaBoostEvtVtxGenerator
MixBoostEvtVtxGenerator
which are used to apply a boost to HepMCProducts for generators that are not configured to produce asymmetric collisions, as is needed for proton-lead.
They correspond to standard and embedded workflows, respectively.
These are currently run in the relval matrix, so I modified wf's 280 and 301 as follows to test them:

cmsDriver.py AMPT_PPb_5020GeV_MinimumBias_cfi --conditions auto:run2_mc --eventcontent RAWSIM -s GEN,SIM --datatier GEN-SIM --beamspot Realistic5TeVPPbBoost

cmsDriver.py Pyquen_DiJet_pt80to120_2760GeV_cfi --conditions auto:run2_mc_hi -s GEN,SIM --pileup_input das:/RelValHydjetQ_MinBias_5020GeV/CMSSW_7_6_0_pre6-76X_mcRun2_HeavyIon_v4-v1/GEN-SIM --era Run2_2016,Run2_HI --eventcontent FEVTDEBUG --scenario HeavyIons --pileup HiMixGEN --datatier GEN-SIM --beamspot Match5TeVPPbBoost

Note that for that for these wf's the vertex smearing parameters get loaded in this package instead of IOMC.EventVertexGenerators which is not so nice. These parameters still correspond to 2013 pPb collisions, but I propose to clean this up in a different PR, once we have a guess for the 2016 pPb beam parameters.

While I was touching this package I also forward ported GenHIEventProducer, which was patched into 75X but never made it into CMSSW8. This will eventually need to be (re)added to heavy ion workflows (as was done in 75X), which will require updating the configBuilder, but I again propose to leave this as separate PR.

I will also make this PR in 80X, as it's not yet clear which release we'll use for pPb GEN-SIM.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 1, 2016

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

It involves the following packages:

GeneratorInterface/HiGenCommon

@covarell, @perrozzi, @thuer, @cmsbuild, @davidlange6, @govoni can you please review it and eventually sign? Thanks.
@cbaus, @agrohsje, @mkirsano this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@mandrenguyen
Copy link
Contributor Author

@davidlange6 davidlange6 merged commit dba7074 into cms-sw:CMSSW_8_1_X Jun 24, 2016
@mandrenguyen mandrenguyen deleted the consumesHiGenCommon81X 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

3 participants