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

CMSSW_11_2_0 dqm:@rerecoCommon requires sim information to run #32634

Closed
davidlange6 opened this issue Jan 12, 2021 · 19 comments
Closed

CMSSW_11_2_0 dqm:@rerecoCommon requires sim information to run #32634

davidlange6 opened this issue Jan 12, 2021 · 19 comments

Comments

@davidlange6
Copy link
Contributor

Running a reco job
cmsDriver.py step3 --conditions auto:phase1_2021_realistic -n -1 --era Run3 --eventcontent AODSIM,MINIAODSIM,DQM --runUnscheduled -s RAW2DIGI,L1Reco,RECO,RECOSIM,EI,PAT,DQM:@rerecoCommon+@miniAODDQM --datatier AODSIM,MINIAODSIM,DQMIO --geometry DB:Extended --
nThreads 8 --filein file:tt_digi_DIGI_DATAMIX_L1_DIGI2RAW_HLT.root --fileout recostep.root --suffix "-j JobReport5.xml " --customise Validation/Performance/TimeMemorySummary.customiseWithTimeMemorySummary

on a RAWSIM input yields

An exception of category 'ProductNotFound' occurred while
[0] Processing Event run: 1 lumi: 1 event: 11 stream: 0
[1] Running path 'dqmoffline_14_step'
[2] Prefetching for module L1TMuonEndCapTrackProducer/'valEmtfStage2Digis'
[3] Prefetching for module GEMPadDigiClusterProducer/'simMuonGEMPadDigiClusters'
[4] Calling method for module GEMPadDigiProducer/'simMuonGEMPadDigis'
Exception Message:
Principal::getByToken: Found zero products matching all criteria
Looking for type: MuonDigiCollection<GEMDetId,GEMDigi>
Looking for module label: simMuonGEMDigis
Looking for productInstanceName:

my understanding is that DQM should not need any sim information (let alone sim information that is not in RAWSIM data tier)

@cmsbuild
Copy link
Contributor

A new Issue was created by @davidlange6 David Lange.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@makortel
Copy link
Contributor

assign dqm

@cmsbuild
Copy link
Contributor

New categories assigned: dqm

@jfernan2,@andrius-k,@fioriNTU,@kmaeshima,@ErnestaP you have been requested to review this Pull request/Issue and eventually sign? Thanks

@jfernan2
Copy link
Contributor

@seungjin-yang @quark2 @hyunyong as GEM DQM contacts, can you please have a look and provide a fix?

@seungjin-yang
Copy link
Contributor

seungjin-yang commented Jan 12, 2021

@jshlee I found some problems.
Firstly, please look at this file. L1Trigger/L1TMuonEndCap/python/simEmtfDigis_cfi.py
We need to replace GEMInput of simEmtfDigisData (L1TMuonEndCapTrackProducer) with the real data versioned one.

But I can’t find GEMPadDigiClusterProducer for real data. L1Trigger/L1TGEM/python/simMuonGEMPadDigiClusters_cfi.py

@jshlee
Copy link
Contributor

jshlee commented Feb 4, 2021

@dildick can you look into this.

@dildick
Copy link
Contributor

dildick commented Feb 4, 2021

yes, I'm working on a fix

@dildick
Copy link
Contributor

dildick commented Feb 4, 2021

@seungjin-yang I'm afraid there is no GEMPadDigiClusterProducer yet for real data; the development the unpacker is still ongoing. What you can do is 1) unpack the GEMDigis (add muonGEMDigis) 2) add the trigger-primitives (simMuonGEMPadDigis, simMuonGEMPadDigiClusters), and add 3) the customization

from Configuration.Eras.Modifier_run3_GEM_cff import run3_GEM
    run3_GEM.toModify(process.simMuonGEMPadDigis,
        InputCollection         = 'muonGEMDigis',
    )

@dildick
Copy link
Contributor

dildick commented Feb 4, 2021

seungjin-yang added a commit to slowmoyang/cmssw that referenced this issue Feb 8, 2021
@seungjin-yang
Copy link
Contributor

@dildick Thank you for your guidance. I made a simple patch. Can you review it? slowmoyang@ca70c5b

@seungjin-yang
Copy link
Contributor

@davidlange6 Can you tell me which matrix or file I should use for testing?

@dildick
Copy link
Contributor

dildick commented Feb 8, 2021

@seungjin-yang it looks ok

@seungjin-yang
Copy link
Contributor

seungjin-yang commented Feb 23, 2021

@davidlange6, @jfernan2 Can you give me workflow number or a test input file for testing patch slowmoyang@ca70c5b ?

@davidlange6
Copy link
Contributor Author

davidlange6 commented Feb 23, 2021 via email

@seungjin-yang
Copy link
Contributor

Thanks!

@dildick
Copy link
Contributor

dildick commented May 11, 2021

@seungjin-yang I'll submit a (modified) PR.

@dildick
Copy link
Contributor

dildick commented Jun 16, 2021

@davidlange6 This issue can be closed.

@jfernan2
Copy link
Contributor

+1
For the records

@cmsbuild
Copy link
Contributor

This issue is fully signed and ready to be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants