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.

Sign up for GitHub

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

Jump to bottom

PCL new workflow for High Perf Beamspot, 2017B relval, 2017B HPBS relval (backport) #19669

Merged
merged 31 commits into from Jul 19, 2017
Merged

PCL new workflow for High Perf Beamspot, 2017B relval, 2017B HPBS relval (backport) #19669

merged 31 commits into from Jul 19, 2017

Conversation

franzoni
Copy link

backport of #19427
and superseeds #19432

Giovanni Franzoni and others added 30 commits July 11, 2017 00:42
adding ppEra_Run2_2017_trackingOnly and options to chose RECO:which_r…
d7be05e 
…eco_sequence
substantiate ppEra_Run2_2017_trackingOnly
870a15c
get RunExpressProcessing.py going
f3eaf82
revert temporarily to full RECO Sequence, because ALCARECOTkAlMinBias…
e3dc097 
…TkAlDQM needs jets
option message for RunAlcaHarvesting.py
60bffcc
customize reconstruction_trackingOnly to run w/ scenario ppEra_Run2_2…
9d4d4a9 
…017_trackingOnly: remove ALCARECOTkAlMinBiasDQM and eventinterpretaion_step
remove crash form EI by modifying cbSc of ppEra_Run2_2017_trackingOnl…
0164126 
…y (and undo previous solution)
set up customisePostEra_Run2_2017_harvesting_trackingOnly to customiz…
451f12f 
…e the labels of the dropboxmetadata
introduce customise_HPbeamspot
3261768
introduce customise_HPbeamspot - clean
f7fb350
PCLMetadataWriter comments
5e46308
remove ALCARECOTkAlMinBiasTkAlDQM because it needs calojets ( not ava…
6093cec 
…ilable when running tracking only reco )
clean customise_HPbeamspot
3c7007f
2017 data relval on HLTPhysics
55e2fba
resolve conflict with 19412
6e2166d
resolve conflict with 19412
5eea36f
run 297227 for 2017B relvals
e7155ec
@rmanzoni
add function to print BS
ed9586b
@sarafiorendi
add possibility to dump txt file in the harvesting step. to be improved
0b51097
@sarafiorendi
add parameters to harvester cfg
16f9ef7
set DumpTxt to true for the ALCAHARVESTBeamSpotByLumi harvester
1023d16
relval2017 menu un 2017B relval
2c37c69
define a default scenario first and then add the era-based modifiers …
6e3f874 
…on top (address comment slava77)
move customise_HPbeamspot to TkAlCaRecoProducers (address comment sla…
8d04598 
…va77)
add return (address comment slava77)
8271d20
@rmanzoni
remove unneeded comments
149faa7
@rmanzoni
remove unneeded comments / 2
2aa7a09
better interface for dumpBeamSpotTxt - credits Sir rmanzoni (address …
d2e5f96 
…comment slava77)
add test commands (address comment slava77)
72306dd
introduce addEI in Reco, factored from scenario name cbSc (address co…
80df377 
…mment slava77)
@franzoni franzoni mentioned this pull request Jul 10, 2017
Closed
@cmsbuild
Copy link
Contributor

+1
Tested at: da371be
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19669/21345/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
82c77dc
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19669/21345/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-19669/21345/git-merge-result

@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-19669/21345/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 0 differences found in the comparisons
  • DQMHistoTests: Total files compared: 22
  • DQMHistoTests: Total histograms compared: 1816233
  • DQMHistoTests: Total failures: 125
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 1815942
  • DQMHistoTests: Total skipped: 166
  • DQMHistoTests: Total Missing objects: 0
  • Checked 90 log files, 14 edm output root files, 22 DQM output files

@slava77
Copy link
Contributor

slava77 commented Jul 12, 2017

+1

for #19669 da371be

@franzoni
Copy link
Author

+1

@fabozzi
Copy link
Contributor

fabozzi commented Jul 13, 2017

+1

@kpedro88
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_9_2_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_9_3_X is complete. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar

@davidlange6
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit 18a89e2 into cms-sw:CMSSW_9_2_X Jul 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
alca-approved backport comparison-available fully-signed operations-approved orp-approved pdmv-approved reconstruction-approved tests-approved upgrade-approved
Projects
None yet
Milestone
CMSSW_9_2_X
Development

Successfully merging this pull request may close these issues.

None yet
8 participants
@franzoni @cmsbuild @slava77 @fabozzi @kpedro88 @davidlange6 @rmanzoni @sarafiorendi