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

Add more Run3 and Phase2 generator fragments #29859

Merged
merged 16 commits into from May 18, 2020

Conversation

akssikdar
Copy link
Contributor

@akssikdar akssikdar commented May 16, 2020

PR description:

This PR append generator fragments for Run3 and Phase2 workflows.

PR includes
1. Added new generator fragments for BuMixing_BMuonFilter_forSTEAM_14TeV_TuneCP5 and Upsilon1SToMuMu_forSTEAM_14TeV_TuneCP5, intended to use for Run3 2021 workflows.
2. Added new generator fragments of TenTau_E_15_500_Eta3p1_pythia8 for Phase2 RelVal production.
3. Added corresponding entries for 1) and 2) in the RelVal submission macros (upgradeWorkflowComponents.py, relval_steps.py)

PR validation:

Tested locally with the related workflows and log file added:

runTheMatrix.py --what upgrade -l 23320.0,23321.0 -t 8 --command='-n 5' >& bumix_upsilon_run5evt.log

runTheMatrix.py --what upgrade -l 23322.0 -t 8 --command='--customise SLHCUpgradeSimulations/Configuration/aging.customise_aging_1000 -n 10' >& tentau_eta3p1_run10evt.log

bumix_upsilon_run5evt.log
tentau_eta3p1_run10evt.log

@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-29859/15418

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @akssikdar (Anup Kumar Sikdar) for master.

It involves the following packages:

Configuration/Generator
Configuration/PyReleaseValidation

@SiewYan, @pgunnell, @efeyazgan, @mkirsano, @chayanit, @wajidalikhan, @kpedro88, @cmsbuild, @GurpreetSinghChahal, @agrohsje, @alberto-sanchez, @qliphy can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @makortel, @fabiocos, @slomeo this is something you requested to watch as well.
@silviodonato, @dpiparo you are the release manager for this.

cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@@ -958,5 +958,8 @@ def __init__(self, howMuch, dataset):
('TauToMuMuMu_14TeV_TuneCP5_cfi', UpgradeFragment(Kby(18939,189393),'TauToMuMuMu_14TeV')), # effi = 5.280e-04
('BdToKstarEleEle_14TeV_TuneCP5_cfi', UpgradeFragment(Kby(206,2061),'BdToKstarEleEle_14TeV')), #effi = 4.850e-02
('ZpTT_1500_14TeV_TuneCP5_cfi', UpgradeFragment(Kby(9,50),'ZpTT_1500_14')),
('BuMixing_BMuonFilter_forSTEAM_14TeV_TuneCP5_cfi', UpgradeFragment(Kby(900,10000),'BuMixing_14')),
('Upsilon1SToMuMu_forSTEAM_14TeV_TuneCP5_cfi', UpgradeFragment(Kby(9,50),'Upsilon1SToMuMu_14')),
('TenTau_15_500_Eta3p1_pythia8_cfi', UpgradeFragment(Kby(9,100),'TenTau_E_15_500_Eta3p1')),

Choose a reason for hiding this comment

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

I was asking to remove _E from 'TenTau_E_15_500_Eta3p1' not from config file name.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is it OK now?

@cmsbuild
Copy link
Contributor

cmsbuild commented May 17, 2020

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/6362/console Started: 2020/05/18 00:43

@cmsbuild
Copy link
Contributor

+1
Tested at: d45fa5a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-992a59/6362/summary.html
CMSSW: CMSSW_11_1_X_2020-05-17-0000
SCRAM_ARCH: slc7_amd64_gcc820

@cmsbuild
Copy link
Contributor

Comparison job queued.

@cmsbuild
Copy link
Contributor

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-992a59/6362/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 0 differences found in the comparisons
  • DQMHistoTests: Total files compared: 35
  • DQMHistoTests: Total histograms compared: 2694466
  • DQMHistoTests: Total failures: 45
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 2694372
  • DQMHistoTests: Total skipped: 49
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 34 files compared)
  • Checked 150 log files, 16 edm output root files, 35 DQM output files

@chayanit
Copy link

+1
@silviodonato it would be nice to have this PR merged in pre8. Thank you

@qliphy
Copy link
Contributor

qliphy commented May 18, 2020

+1

@kpedro88
Copy link
Contributor

+upgrade

@cmsbuild
Copy link
Contributor

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

@cmsbuild cmsbuild merged commit 3c5534b into cms-sw:master May 18, 2020
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