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

[NOT TO MERGE] PR test DD4hep in 14_0 #45513

Open
wants to merge 2 commits into
base: CMSSW_14_0_X
Choose a base branch
from
Open

[NOT TO MERGE] PR test DD4hep in 14_0 #45513

wants to merge 2 commits into from

Conversation

srimanob
Copy link
Contributor

PR description:

Just for PR test, not to merge.

PR validation:

Nothing.

@cmsbuild cmsbuild added this to the CMSSW_14_0_X milestone Jul 19, 2024
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @srimanob for CMSSW_14_0_X.

It involves the following packages:

  • Configuration/PyReleaseValidation (upgrade, pdmv)

@AdrianoDee, @cmsbuild, @kskovpen, @miquork, @srimanob, @subirsarkar, @sunilUIET can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @fabiocos, @makortel, @missirol, @slomeo this is something you requested to watch as well.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

cmsbuild commented Jul 19, 2024 •
edited
Loading

cms-bot internal usage

@srimanob
Copy link
Contributor Author

test parameters:

  • workflows = 24834.911,29634.911
  • relvals_opt = --what cleanedupgrade,standard,highstats,pileup,generator,extendedgen,production,identity,ged,machine,premix,nano,gpu,2017,2026

@srimanob
Copy link
Contributor Author

@cmsbuild please test

@cmsbuild
Copy link
Contributor

Pull request #45513 was updated. @AdrianoDee, @Dr15Jones, @bsunanda, @civanch, @cmsbuild, @kskovpen, @makortel, @mdhildreth, @miquork, @srimanob, @subirsarkar, @sunilUIET can you please check and sign again.

@srimanob
Copy link
Contributor Author

@cmsbuild please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: RelVals
Size: This PR adds an extra 20KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7b5b73/40500/summary.html
COMMIT: f392cf3
CMSSW: CMSSW_14_0_X_2024-07-19-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/45513/40500/install.sh to create a dev area with all the needed externals and cmssw changes.

RelVals

----- Begin Fatal Exception 19-Jul-2024 21:28:13 CEST-----------------------
An exception of category 'BtlSimHitsValidation' occurred while
   [0] Processing  Event run: 1 lumi: 1 event: 1 stream: 0
   [1] Running path 'validation_step10'
   [2] Calling method for module BtlSimHitsValidation/'btlSimHitsValid'
Exception Message:
GeographicalID: 62800010 (0) is invalid!
----- End Fatal Exception -------------------------------------------------

@srimanob
Copy link
Contributor Author

test parameters:

@srimanob
Copy link
Contributor Author

test parameters:

  • workflows = 24834.911
  • relvals_opt = --what cleanedupgrade,standard,highstats,pileup,generator,extendedgen,production,identity,ged,machine,premix,nano,gpu,2017,2026

@srimanob
Copy link
Contributor Author

@cmsbuild please test

@cmsbuild
Copy link
Contributor

+1

Size: This PR adds an extra 20KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7b5b73/40503/summary.html
COMMIT: f392cf3
CMSSW: CMSSW_14_0_X_2024-07-19-2300/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/45513/40503/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

@fabiocos
Copy link
Contributor

@srimanob in order to fix #45513 (comment) you'll need #45211 (actually just the genuine fix part of it)

@srimanob
Copy link
Contributor Author

Hi @fabiocos
Thx. I tried last night by backport only Geometry/MTDCommonData/src/BTLNumberingScheme.cc, but I see a lot of warning, and I didn't go through it. Anything else apart from that file?

%MSG-w MTDGeom:  (NoModuleName) 20-Jul-2024 00:26:02 CEST pre-events
BTLNumberingScheme::getUnitID(): ****************** Bad RU name, Volume Name = btl:ReadoutUnit

@fabiocos
Copy link
Contributor

@srimanob I have just tried to integrate on top of CMSSW_14_0_X_2024-07-21-0000 this PR and the BTLNumberingScheme.cc update, and it run smoothly.

@fabiocos
Copy link
Contributor

you may check the output on cmsdev45 , in

/build/fabiocos/140X/CMSSW_14_0_X_2024-07-21-0000/work/29634.911_TTbar_14TeV+2026D110_DD4hep

@cmsbuild
Copy link
Contributor

Pull request #45513 was updated. @AdrianoDee, @Dr15Jones, @bsunanda, @civanch, @cmsbuild, @kskovpen, @makortel, @mdhildreth, @miquork, @srimanob, @subirsarkar, @sunilUIET can you please check and sign again.

@srimanob srimanob closed this Jul 24, 2024
@srimanob srimanob deleted the 140_PRTestDD4Hep branch July 24, 2024 17:57
@fabiocos fabiocos mentioned this pull request Jul 29, 2024
Merged
@srimanob srimanob restored the 140_PRTestDD4Hep branch July 30, 2024 07:15
@srimanob srimanob reopened this Jul 30, 2024
@srimanob
Copy link
Contributor Author

test parameters:

@srimanob
Copy link
Contributor Author

test parameters:

@srimanob
Copy link
Contributor Author

test parameters:

@srimanob
Copy link
Contributor Author

@cmsbuild please test

@cmsbuild
Copy link
Contributor

+1

Size: This PR adds an extra 12KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7b5b73/40686/summary.html
COMMIT: 00fd362
CMSSW: CMSSW_14_0_X_2024-07-29-2300/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/45513/40686/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

There are some workflows for which there are errors in the baseline:
29634.911 step 3
The results for the comparisons for these workflows could be incomplete
This means most likely that the IB is having errors in the relvals.The error does NOT come from this pull request

Summary:

  • You potentially removed 117 lines from the logs
  • ROOTFileChecks: Some differences in event products or their sizes found
  • Reco comparison results: 129 differences found in the comparisons
  • DQMHistoTests: Total files compared: 48
  • DQMHistoTests: Total histograms compared: 3343220
  • DQMHistoTests: Total failures: 2565
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 3340635
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 47 files compared)
  • Checked 205 log files, 169 edm output root files, 48 DQM output files
  • TriggerResults: no differences found

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
geometry-pending orp-pending pdmv-pending pending-signatures requires-external tests-approved upgrade-pending
Projects
None yet
Milestone
CMSSW_14_0_X
Development

Successfully merging this pull request may close these issues.
3 participants
@srimanob @cmsbuild @fabiocos