Skip to content

Update reference files for new ETL format with extended module field#4

Merged
cmsbuild merged 1 commit intocms-data:masterfrom
fabiocos:fc-newETLDetId
Sep 24, 2020
Merged

Update reference files for new ETL format with extended module field#4
cmsbuild merged 1 commit intocms-data:masterfrom
fabiocos:fc-newETLDetId

Conversation

@fabiocos
Copy link
Contributor

New reference for updated ETLDetId code

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @fabiocos (Fabio Cossutti) for branch master.

@cmsbuild, @smuzaffar, @mrodozov can you please review it and eventually sign? Thanks.
cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 18, 2020

The tests are being triggered in jenkins.
Tested with other pull request(s) cms-sw/cmssw#31511

@cmsbuild
Copy link
Contributor

+1
Tested at: a4acb58
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5f4010/9432/summary.html
CMSSW: CMSSW_11_2_X_2020-09-18-1100
SCRAM_ARCH: slc7_amd64_gcc820

@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-5f4010/9432/summary.html

Comparison Summary:

  • No significant changes to the logs found
  • Reco comparison results: 593 differences found in the comparisons
  • DQMHistoTests: Total files compared: 35
  • DQMHistoTests: Total histograms compared: 2540471
  • DQMHistoTests: Total failures: 1920
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 2538529
  • DQMHistoTests: Total skipped: 22
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 34 files compared)
  • Checked 149 log files, 22 edm output root files, 35 DQM output files

@silviodonato
Copy link

merge
(new files)

@cmsbuild cmsbuild merged commit 573baa1 into cms-data:master Sep 24, 2020
@fabiocos fabiocos deleted the fc-newETLDetId branch May 19, 2021 12:50
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.

3 participants