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

[G4] Try tag 10.7.2 against cmsdist/master #7040

Closed
wants to merge 1 commit into from

Conversation

mrodozov
Copy link
Contributor

as described in:
cms-externals/geant4#53

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @mrodozov (Mircho Rodozov) for branch IB/CMSSW_12_0_X/master.

@cmsbuild, @smuzaffar, @mrodozov, @iarspider can you please review it and eventually sign? Thanks.
@silviodonato, @dpiparo, @qliphy you are the release manager for this.
cms-bot commands are listed here

@mrodozov
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: RelVals-INPUT
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fc509d/16083/summary.html
COMMIT: deda2a2
CMSSW: CMSSW_12_0_X_2021-06-17-1100/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmsdist/7040/16083/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fc509d/16083/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-fc509d/16083/git-merge-result

RelVals-INPUT

  • 136.862136.862_RunEGamma2018B+RunEGamma2018B+HLTDR2_2018+RECODR2_2018reHLT_skimEGamma_Offline_L1TEgDQM+HARVEST2018_L1TEgDQM/step2_RunEGamma2018B+RunEGamma2018B+HLTDR2_2018+RECODR2_2018reHLT_skimEGamma_Offline_L1TEgDQM+HARVEST2018_L1TEgDQM.log

Comparison Summary

Summary:

  • No significant changes to the logs found
  • Reco comparison results: 51707 differences found in the comparisons
  • DQMHistoTests: Total files compared: 38
  • DQMHistoTests: Total histograms compared: 2871916
  • DQMHistoTests: Total failures: 296001
  • DQMHistoTests: Total nulls: 250
  • DQMHistoTests: Total successes: 2575643
  • DQMHistoTests: Total skipped: 22
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.048 KiB( 37 files compared)
  • DQMHistoSizes: changed ( 10224.0 ): 0.234 KiB SiStrip/MechanicalView
  • DQMHistoSizes: changed ( 250202.181 ): -0.182 KiB SiStrip/MechanicalView
  • DQMHistoSizes: changed ( 312.0 ): -0.004 KiB MessageLogger/Warnings
  • Checked 160 log files, 37 edm output root files, 38 DQM output files
  • TriggerResults: found differences in 12 / 37 workflows

@mrodozov
Copy link
Contributor Author

@civanch I see a lot of reco diffs + one failure not sure yet from what (we'll have to figure that out)
we have the G4_vecgeom IB now, I suggest I close this as the results are available and we continue on the G4_vecgeom branch

@mrodozov
Copy link
Contributor Author

see #7041

@mrodozov mrodozov closed this Jun 18, 2021
@smuzaffar smuzaffar deleted the get-geant-1072 branch June 25, 2021 05:15
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

2 participants