Missing Parameter in L1TTauOffline/'l1tTauOfflineDQM'

[chayanit]

I have been running premixing PU RelVals 2021 scenario to be validated for closed cycle of 11_1_0 and getting this error in production. "workingsPoints" parameter is missing at L1T.

Fatal Exception (Exit code: 8009)
An exception of category 'Configuration' occurred while
[0] Processing Event run: 1 lumi: 148 event: 14705 stream: 0
[1] Calling method for module L1TTauOffline/'l1tTauOfflineDQM'
Exception Message:
MissingParameter: Parameter 'workingsPoints' not found.

See https://dmytro.web.cern.ch/dmytro/cmsprodmon/workflows.php?prep_id=CMSSW_11_1_0_pre8__fullsim_PU2021_premix-1592492510-ZMM_14

It seems something related to this merge PR #28417 @swozniewski. Could you help checking? Thanks in advance.

FYI. @silviodonato

[cmsbuild]

A new Issue was created by @chayanit Chayanit Asawatangtrakuldee.

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

cms-bot commands are listed here

[swozniewski]

Hi, this seems to be a stupid typo. sorry for that! This line is calling 'workingsPoints' (note the 's' in the middle, which I only realized after reading it several times now...) from provenance of the antiElectronID, which has 'workingPoints' in its config. See
https://github.com/cms-tau-pog/cmssw/blob/e3ee2f54d22194b79a0f8ad58d32cfb1b8b946e2/DQMOffline/L1Trigger/src/L1TTauOffline.cc#L681
and
https://github.com/cms-sw/cmssw/blob/master/RecoTauTag/Configuration/python/HPSPFTaus_cff.py#L287
Could you please check whether your workflow runs successfully if you remove the 's' in the middle? I will set up the PR then to fix it.

[chayanit]

Thanks! @swozniewski ok I will resubmit and let you know. It can be noticed only we submitted large production while small test locally or IB tests didn't catch this :(

[makortel]

assign dqm

[cmsbuild]

New categories assigned: dqm

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

[chayanit]

@swozniewski I can't fix this locally. Please make PR and I can only check the production when PR is merged to new release.

[swozniewski]

Ok, sure. I think the problem here is rather clear. Would have only been nice to see that nothing else pops up.

[swozniewski]

@chayanit just in case you did not notice anyway: The two PRs mentioned above are now merged.

[chayanit]

Thanks! indeed I did not notice but now I do :) @swozniewski
I will try another round with new release before I close this issue.

[jfernan2]

+1

[cmsbuild]

This issue is fully signed and ready to be closed.