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

Tau DQM miniAOD validation for DATA #44029

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

gsaha009
Copy link
Contributor

@gsaha009 gsaha009 commented Feb 20, 2024

PR description:

Following the last PR #41995, the miniAOD validation on DATA is prepared for Run3. Twiki page: https://twiki.cern.ch/twiki/bin/viewauth/CMS/SWGuidePFTauIDValidation

PR validation:

  • PR runs fine in local pipeline, and even with workflows.
  • The main validation sequence is divided into two: for MC and Data. The MC sequence is plugged in the validation and the DATA is in the DQM.
  • Tests done with workflow: runTheMatrix.py -l limited -i all --ibeos

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for: NA

Thanks @sroychow

@cmsbuild
Copy link
Contributor

cmsbuild commented Feb 20, 2024

cms-bot internal usage

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44029/38962

  • This PR adds an extra 40KB to repository

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @gsaha009 for master.

It involves the following packages:

  • Validation/RecoTau (dqm)

@tjavaid, @antoniovagnerini, @cmsbuild, @nothingface0, @rvenditti, @syuvivida can you please review it and eventually sign? Thanks.
@missirol this is something you requested to watch as well.
@rappoccio, @antoniovilela, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@tjavaid
Copy link

tjavaid commented Feb 20, 2024

please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: RelVals
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a77f16/37584/summary.html
COMMIT: 1f73cfc
CMSSW: CMSSW_14_1_X_2024-02-20-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/44029/37584/install.sh to create a dev area with all the needed externals and cmssw changes.

RelVals

----- Begin Fatal Exception 20-Feb-2024 17:21:07 CET-----------------------
An exception of category 'ProductNotFound' occurred while
   [0] Processing  Event run: 1 lumi: 1 event: 1 stream: 0
   [1] Running path 'dqmoffline_step'
   [2] Calling method for module TauValPatJetSelector/'kinematicSelectedPFJets'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for type: std::vector<pat::Jet>
Looking for module label: slimmedJets
Looking for productInstanceName: 

   Additional Info:
      [a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.

----- End Fatal Exception -------------------------------------------------
----- Begin Fatal Exception 20-Feb-2024 17:21:09 CET-----------------------
An exception of category 'ProductNotFound' occurred while
   [0] Processing  Event run: 1 lumi: 1 event: 1 stream: 0
   [1] Running path 'dqmoffline_step'
   [2] Calling method for module TauValPatJetSelector/'kinematicSelectedPFJets'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for type: std::vector<pat::Jet>
Looking for module label: slimmedJets
Looking for productInstanceName: 

   Additional Info:
      [a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.

----- End Fatal Exception -------------------------------------------------

@tjavaid
Copy link

tjavaid commented May 16, 2024

please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: RelVals
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a77f16/39396/summary.html
COMMIT: 1f73cfc
CMSSW: CMSSW_14_1_X_2024-05-15-2300/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/44029/39396/install.sh to create a dev area with all the needed externals and cmssw changes.

RelVals

----- Begin Fatal Exception 16-May-2024 05:22:00 CEST-----------------------
An exception of category 'ProductNotFound' occurred while
   [0] Processing  Event run: 1 lumi: 1 event: 1 stream: 0
   [1] Running path 'dqmoffline_step'
   [2] Calling method for module TauValPatJetSelector/'kinematicSelectedPFJets'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for type: std::vector<pat::Jet>
Looking for module label: slimmedJets
Looking for productInstanceName: 

   Additional Info:
      [a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.

----- End Fatal Exception -------------------------------------------------
----- Begin Fatal Exception 16-May-2024 05:22:03 CEST-----------------------
An exception of category 'ProductNotFound' occurred while
   [0] Processing  Event run: 1 lumi: 1 event: 1 stream: 0
   [1] Running path 'dqmoffline_step'
   [2] Calling method for module TauValPatJetSelector/'kinematicSelectedPFJets'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for type: std::vector<pat::Jet>
Looking for module label: slimmedJets
Looking for productInstanceName: 

   Additional Info:
      [a] If you wish to continue processing events after a ProductNotFound exception,
add "TryToContinue = cms.untracked.vstring('ProductNotFound')" to the "options" PSet in the configuration.

----- End Fatal Exception -------------------------------------------------

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44029/40680

  • This PR adds an extra 24KB to repository

@cmsbuild
Copy link
Contributor

Pull request #44029 was updated. @antoniovilela, @rvenditti, @davidlange6, @antoniovagnerini, @syuvivida, @tjavaid, @rappoccio, @cmsbuild, @fabiocos, @nothingface0 can you please check and sign again.

@sroychow
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: UnitTests
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a77f16/40024/summary.html
COMMIT: 9e45bf6
CMSSW: CMSSW_14_1_X_2024-06-23-0000/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/44029/40024/install.sh to create a dev area with all the needed externals and cmssw changes.

Unit Tests

I found 1 errors in the following unit tests:

---> test TestDQMOfflineConfigurationGotAll had ERRORS

Comparison Summary

Summary:

  • You potentially added 270 lines to the logs
  • ROOTFileChecks: Some differences in event products or their sizes found
  • Reco comparison results: 3 differences found in the comparisons
  • DQMHistoTests: Total files compared: 48
  • DQMHistoTests: Total histograms compared: 3343324
  • DQMHistoTests: Total failures: 6
  • DQMHistoTests: Total nulls: 911
  • DQMHistoTests: Total successes: 3342387
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 113342.45 KiB( 47 files compared)
  • DQMHistoSizes: changed ( 10224.0,... ): 3356.640 KiB RecoTauV/miniAODValidation
  • DQMHistoSizes: changed ( 10224.0,... ): -0.113 KiB MessageLogger/Errors
  • DQMHistoSizes: changed ( 10224.0,... ): -0.113 KiB MessageLogger/Warnings
  • DQMHistoSizes: changed ( 13234.0,... ): 3575.469 KiB RecoTauV/miniAODValidation
  • DQMHistoSizes: changed ( 136.7611,... ): 2921.013 KiB RecoTauV/miniAODValidation
  • Checked 202 log files, 165 edm output root files, 48 DQM output files
  • TriggerResults: found differences in 21 / 46 workflows

@gsaha009
Copy link
Contributor Author

gsaha009 commented Jun 24, 2024

Hi @danielwinterbottom
Can you please have a look? Thanks

@gsaha009
Copy link
Contributor Author

gsaha009 commented Jun 24, 2024

Update: This workflow 25202.0 failed for other CMSSW versions as well, even without the changes for TauDQM made in this PR @cmsbuild
The error in wf 25202.0 only is the following:

DIGI:pdigi_valid,L1,DIGI2RAW,HLT:@relval2016,ENDJOB
We have determined that this is simulation (if not, rerun cmsDriver.py with --data)
entry file:step1.root
the query is file dataset = /RelValMinBias_13/CMSSW_10_6_0-106X_mcRun2_asymptotic_v3-v1/GEN-SIM
DAS succeeded after 1 attempts 0
found files:  []
Step: DIGI Spec: ['pdigi_valid']
Step: L1 Spec: 
Step: DIGI2RAW Spec: 
Step: HLT Spec: ['@relval2016']
Step: ENDJOB Spec: 
customising the process with customizeHLTforMC from HLTrigger/Configuration/customizeHLTforMC
Starting  cmsRun  step2_DIGI_L1_DIGI2RAW_HLT_PU.py
2024-06-24 17:06:32.769201: I tensorflow/core/util/port.cc:110] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
24-Jun-2024 17:06:34 CEST  Initiating request to open file file:step1.root
24-Jun-2024 17:06:34 CEST  Successfully opened file file:step1.root
24-Jun-2024 17:06:38 CEST  Closed file file:step1.root
----- Begin Fatal Exception 24-Jun-2024 17:06:38 CEST-----------------------
An exception of category 'NoSecondaryFiles' occurred while
   [0] Constructing the EventProcessor
   [1] Constructing module: class=MixingModule label='mix'
Exception Message:
RootEmbeddedFileSequence no input files specified for secondary input source.
----- End Fatal Exception -------------------------------------------------

and this is not because of the changes made in this PR

@gsaha009
Copy link
Contributor Author

gsaha009 commented Jul 11, 2024

@gsaha009
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

Milestone for this pull request has been moved to CMSSW_14_2_X. Please open a backport if it should also go in to CMSSW_14_1_X.

@antoniovilela
Copy link
Contributor

ping (to make bot change milestone)

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.

5 participants