search

cms-sw / cmssw

CMS Offline Software
http://cms-sw.github.io/
Apache License 2.0
1.05k stars 4.23k forks source link

Noisy `L1NNCaloTauProducer` in Phase-2 workflows #44766

Open mmusich opened 1 month ago

mmusich commented 1 month ago

Phase-2 workflow step2 logs are full of:

** WARNING : FOUND 1902 TOWERS WITH towerIeta=-1016 AND towerIphi=-962
%MSG
%MSG-w BrokenTowers:  L1NNCaloTauProducer:l1tNNCaloTauProducer  17-Apr-2024 21:07:07 CEST Run: 1 Event: 5
 ** WARNING : FOUND 1902 TOWERS WITH towerIeta=-1016 AND towerIphi=-962
%MSG
Begin processing the 6th record. Run 1, Event 6, LumiSection 1 on stream 0 at 17-Apr-2024 21:07:08.558 CEST
%MSG-w BrokenTowers:  L1NNCaloTauEmulator:l1tNNCaloTauEmulator  17-Apr-2024 21:07:12 CEST Run: 1 Event: 6
 ** WARNING : FOUND 1682 TOWERS WITH towerIeta=-1016 AND towerIphi=-962
%MSG
%MSG-w BrokenTowers:  L1NNCaloTauProducer:l1tNNCaloTauProducer  17-Apr-2024 21:07:12 CEST Run: 1 Event: 6
 ** WARNING : FOUND 1682 TOWERS WITH towerIeta=-1016 AND towerIphi=-962
%MSG

example 24834.0.

Please fix.

cmsbuild commented 1 month ago

cms-bot internal usage

cmsbuild commented 1 month ago

A new Issue was created by @mmusich.

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

cms-bot commands are listed here

makortel commented 1 month ago

assign l1,upgrade

cmsbuild commented 1 month ago

New categories assigned: l1,upgrade

@epalencia,@aloeliger,@srimanob,@subirsarkar you have been requested to review this Pull request/Issue and eventually sign? Thanks

aloeliger commented 1 month ago

@jonamotta I believe out of all the tau projects, this one is yours.

jonamotta commented 1 month ago

Yes indeed, that's mine. I have just opened PR#44770 which fixes it by putting the output under the DEBUG flag.