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

[backport] MTD geometry and reconstruction: synchronise with master up to PR 33762 #33779

Merged
merged 25 commits into from May 19, 2021

Conversation

fabiocos and others added 25 commits May 19, 2021 14:26
@fabiocos
Copy link
Contributor Author

backport

@silviodonato
Copy link
Contributor

merge

@cmsbuild cmsbuild merged commit e60153e into cms-sw:CMSSW_11_3_X May 19, 2021
@cmsbuild
Copy link
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-496206/15185/summary.html
COMMIT: 5eb2592
CMSSW: CMSSW_11_3_X_2021-05-19-1100/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/33779/15185/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • No significant changes to the logs found
  • Reco comparison results: 102 differences found in the comparisons
  • DQMHistoTests: Total files compared: 38
  • DQMHistoTests: Total histograms compared: 2877046
  • DQMHistoTests: Total failures: 365
  • DQMHistoTests: Total nulls: 1
  • DQMHistoTests: Total successes: 2876658
  • DQMHistoTests: Total skipped: 22
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: -0.004 KiB( 37 files compared)
  • DQMHistoSizes: changed ( 312.0 ): -0.004 KiB MessageLogger/Warnings
  • Checked 160 log files, 37 edm output root files, 38 DQM output files
  • TriggerResults: no differences found

@jfernan2
Copy link
Contributor

+1
for the records...

@srimanob
Copy link
Contributor

+Upgrade

This is needed for 11_3 Phase-2 (small) production.

@silviodonato
Copy link
Contributor

I'm sorry I merged the wrong PR... I reverted this PR with #33793. The morning IB should be ok

@fabiocos
Copy link
Contributor Author

I'll propose this PR again to go through the usual review, please recall to revert also the corresponding cmsdist update otherwise I expect some unit test to fail in 11_3_X (not a big drama, but)

@fabiocos fabiocos deleted the fc-bp33598 branch May 20, 2021 08:05
fabiocos added a commit to fabiocos/cmssw that referenced this pull request May 20, 2021
…se with master up to PR 33762""

This reverts commit f5281cb, restoring PR cms-sw#33779
ianna pushed a commit to ianna/cmssw that referenced this pull request Jun 16, 2021
…se with master up to PR 33762""

This reverts commit f5281cb, restoring PR cms-sw#33779
@jfernan2
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