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

tracking specific relval IB test workflows #14540

Merged

Conversation

hengne
Copy link
Contributor

@hengne hengne commented May 17, 2016

[This PR is made on top of PR #14469 (normal relval matrix update, and turn on EI and PAT steps in upgrade 2017 workflows]

Adding the tracking specific IB test TTbar workflows to study the tracking algorithms with low PU with different tracking configurations, that developed by Matti @makortel
Those workflows are expected to be only used in IB test, and local run, not for relval production, because they are repeated TTbar workflows that writing to the same output dataset names (same as the default TTbar one.).

Those IB test workflows are:

2016 fullSim:
1325.1_TTbar_13+TTbar_13INPUT+DIGIUP15+RECOUP15_trackingOnly+HARVESTUP15_trackingOnly
1325.2_TTbar_13+TTbar_13INPUT+DIGIUP15+RECOUP15_trackingLowPU+HARVESTUP15
1325.3_TTbar_13+TTbar_13INPUT+DIGIUP15+RECOUP15_trackingOnlyLowPU+HARVESTUP15_trackingOnly

2016 fastSim:
135.3_SingleMuPt100_UP15+SingleMuPt100FS_UP15+HARVESTUP15FS+MINIAODMCUP15FS

2017 fullSim:
10024.1_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFullINPUT+DigiFull_2017+RecoFull_trackingOnly_2017+HARVESTFull_trackingOnly_2017
10024.2_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFullINPUT+DigiFull_2017+RecoFull_trackingRun2_2017+HARVESTFull_2017
10024.3_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFullINPUT+DigiFull_2017+RecoFull_trackingOnlyRun2_2017+HARVESTFull_trackingOnly_2017
10024.4_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFullINPUT+DigiFull_2017+RecoFull_trackingPhase1PU70_2017+HARVESTFull_2017
10024.5_TTbar_13+TTbar_13TeV_TuneCUETP8M1_2017_GenSimFullINPUT+DigiFull_2017+RecoFull_trackingOnlyPhase1PU70_2017+HARVESTFull_trackingOnly_2017

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @hengne (Hengne Li) for CMSSW_8_1_X.

It involves the following packages:

Configuration/PyReleaseValidation

@cmsbuild, @srimanob, @davidlange6, @hengne, @fabozzi can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald 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

@hengne
Copy link
Contributor Author

hengne commented May 17, 2016

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented May 17, 2016

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

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@hengne
Copy link
Contributor Author

hengne commented May 18, 2016

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 2, 2016

Pull request #14540 was updated. @cmsbuild, @srimanob, @davidlange6, @hengne, @fabozzi can you please check and sign again.

@hengne
Copy link
Contributor Author

hengne commented Jun 2, 2016

@makortel @davidlange6 rebased, and local tested to be all OK. Sorry for the delay on this task because of some urgent analysis tasks early this week...

@hengne
Copy link
Contributor Author

hengne commented Jun 2, 2016

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 2, 2016

@hengne
Copy link
Contributor Author

hengne commented Jun 2, 2016

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 2, 2016

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

@hengne
Copy link
Contributor Author

hengne commented Jun 2, 2016

@davidlange6 this PR is ready.

wmsplit['RECODR2_2016sreHLT']=1
wmsplit['RECODR2_50nsreHLT']=5
wmsplit['RECODR2_25nsreHLT']=5
wmsplit['RECODR2_2016reHLT']=5
Copy link
Contributor

Choose a reason for hiding this comment

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

its beginning to look like these things should become properties of the workflows themselves rather than having a long list in MatrixInjector (which should know much about specific workflows)
@hengne

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@davidlange6 good idea! let me think how to implement this idea after i finished the tasks for reHLT. i will either just implement it, or will come back to in case i see a show stopper.

@davidlange6
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit 051bc7a into cms-sw:CMSSW_8_1_X Jun 2, 2016
@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 2, 2016

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