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

Override split level in miniaod 92X version #20037

Merged
merged 3 commits into from Aug 23, 2017

Conversation

arizzi
Copy link
Contributor

@arizzi arizzi commented Aug 3, 2017

This is the same of #19680, but for 92X

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 3, 2017

A new Pull Request was created by @arizzi for CMSSW_9_2_X.

It involves the following packages:

Configuration/Applications
PhysicsTools/PatAlgos

@perrotta, @monttj, @cmsbuild, @franzoni, @slava77, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @mmarionncern, @gouskos, @rappoccio, @imarches, @ahinzmann, @acaudron, @gpetruc, @TaiSakuma, @Martin-Grunewald, @jdolen, @nhanvtran, @JyothsnaKomaragiri, @gkasieczka, @schoef, @ferencek, @mverzett, @mariadalfonso, @pvmulder, @seemasharmafnal this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

@slava77
Copy link
Contributor

slava77 commented Aug 3, 2017

this has no utility for 92X: there are no new features useful for physics studies etc.

@slava77
Copy link
Contributor

slava77 commented Aug 3, 2017

@cmsbuild please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 3, 2017

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/22069/console Started: 2017/08/03 19:12

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 3, 2017

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 3, 2017

Comparison job queued.

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 3, 2017

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

Comparison Summary:

  • No significant changes to the logs found
  • ROOTFileChecks: Some differences in event products or their sizes found
  • Reco comparison results: 0 differences found in the comparisons
  • DQMHistoTests: Total files compared: 22
  • DQMHistoTests: Total histograms compared: 1791808
  • DQMHistoTests: Total failures: 44290
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 1747352
  • 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 Aug 4, 2017

backport of #19680

@slava77
Copy link
Contributor

slava77 commented Aug 4, 2017

+1

for #20037 2312220

  • somewhat trivial backport: the same topic branch/same commit is used as the recently signed off (not yet merged as of Aug 3) version in 93X
  • jenkins tests pass

this PR only changes the stream out settings and should be straightforward to apply in the MCv1 and the prompt reco.

@davidlange6
Copy link
Contributor

merge

@cmsbuild cmsbuild merged commit 4258145 into cms-sw:CMSSW_9_2_X Aug 23, 2017
@arizzi arizzi deleted the overrideSplitMiniAOD branch October 11, 2017 08:55
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