Noisy PileupJetIdProducer in Prompt reconstruction

[makortel]

While investigating a PromptReco reproducer in #45089 (comment) I noticed many printouts along

%MSG-e FactorizedJetCorrectorCalculator:   PileupJetIdProducer:pileupJetIdCalculatorPUPPIDQM 29-May-2024 23:36:50 CEST  Run: 381067 Event: 682965806
jet phi is not set
%MSG

from pileupJetIdCalculatorPUPPIDQM and pileupJetIdCalculatorCHSDQM. In that example job, in 26466 events there were total of 4733863 printouts, corresponding to 767 MB of log.

Could these be silenced in some way?

[makortel]

assign reconstruction, dqm

[cmsbuild]

New categories assigned: reconstruction,dqm

@jfernan2,@mandrenguyen,@rvenditti,@syuvivida,@tjavaid,@nothingface0,@antoniovagnerini you have been requested to review this Pull request/Issue and eventually sign? Thanks

[cmsbuild]

cms-bot internal usage

[cmsbuild]

A new Issue was created by @makortel.

@Dr15Jones, @antoniovilela, @rappoccio, @sextonkennedy, @makortel, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[makortel]

Could somebody look into this? The amount of logs is quite substantial.

[jfernan2]

type jetmet

[jfernan2]

@iatakisi as current DQM Offline expert, could you please have a look?

[nurfikri89]

@makortel @jfernan2 has this been resolved? I can take a look at this from the jetmet side.

[jfernan2]

No, it has not, please have a llok and make a proposal for solution. Thanks

[nurfikri89]

The printouts appear because one of the levels in the latest JECs is now dependent on jet phi. The variable is not explicitly set here:

cmssw/RecoJets/JetProducers/plugins/PileupJetIdProducer.cc

Lines 158 to 165 in 77c6c38

	if (ispat) {
	jecCor_->setJetPt(patjet->correctedJet(0).pt());
	} else {
	jecCor_->setJetPt(jet.pt());
	}
	jecCor_->setJetEta(jet.eta());
	jecCor_->setJetA(jet.jetArea());
	jecCor_->setRho(rho);

Simply adding jecCor_->setJetPhi(jet.phi()); suppress the message. I have tested that the prinouts no longer appear using RunPromptReco.py. The PRs for the master and 14_0_X branches will be submitted shortly.

[jfernan2]

Thanks @nurfikri89 !

[jfernan2]

+1
Fixed by #45270