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

PPS: direct simulation update A #30466

Merged
merged 8 commits into from Jul 7, 2020
Merged

Conversation

jan-kaspar
Copy link
Contributor

PR description:

Several updates to the "direct" simulation of PPS have been presented in the PPS General Meeting:
https://indico.cern.ch/event/927936/contributions/3909004/attachments/2057769/3451230/kaspar_GM_direct_simu.pdf
This PR contains a first part of them (thus the A in the name). Others will follow once the Run3 geometry and RP layout are in place, cf. #30252.

This PR contains the following updates

The "direct" simulation is not used in any central workflow, thus no change in these WFs is expected.

PR validation:

Several validation plots are available in the PPS GM talk linked in the beginning.

@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-30466/16622

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @jan-kaspar for master.

It involves the following packages:

CalibPPS/ESProducers
CondFormats/PPSObjects
Configuration/Eras
IOMC/EventVertexGenerators
Validation/CTPPS

@andrius-k, @kmaeshima, @schneiml, @christopheralanwest, @civanch, @mdhildreth, @cmsbuild, @silviodonato, @franzoni, @jfernan2, @fioriNTU, @tlampen, @tocheng, @pohsun, @fabiocos, @davidlange6 can you please review it and eventually sign? Thanks.
@makortel, @forthommel, @tocheng, @Martin-Grunewald, @mmusich, @fabiocos, @seemasharmafnal this is something you requested to watch as well.
@silviodonato, @dpiparo you are the release manager for this.

cms-bot commands are listed here

@jfernan2
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 30, 2020

The tests are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

+1
Tested at: 83c4850
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6c73c4/7525/summary.html
CMSSW: CMSSW_11_2_X_2020-06-29-2300
SCRAM_ARCH: slc7_amd64_gcc820

@cmsbuild
Copy link
Contributor

Comparison job queued.

@jfernan2
Copy link
Contributor

+1

@tlampen
Copy link
Contributor

tlampen commented Jul 1, 2020

+1

@civanch
Copy link
Contributor

civanch commented Jul 2, 2020

+1

@jan-kaspar
Copy link
Contributor Author

@davidlange6, @fabiocos, @franzoni, @silviodonato: could you please check (and possibly sign off) the small change in Era_Run3_cff.py? It simply replaces the inapplicable ctpps_2018 (Run2) with more appropriate ctpps_2021. Many thanks in advance!

@silviodonato
Copy link
Contributor

+operations

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 7, 2020

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo (and backports should be raised in the release meeting by the corresponding L2)

@silviodonato
Copy link
Contributor

+1

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

6 participants