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

P2GT Tau isolation broken in menu #1237

Open
artlbv opened this issue Apr 2, 2024 · 2 comments
Open

P2GT Tau isolation broken in menu #1237

artlbv opened this issue Apr 2, 2024 · 2 comments

Comments

@artlbv
Copy link

artlbv commented Apr 2, 2024

See here:

#1225 (comment)

Hi there @Duchstf @HaarigerHarald @qvyz, it seems to me that this PR introduced this error:

----- Begin Fatal Exception 02-Apr-2024 13:17:39 CEST-----------------------
An exception of category 'StdException' occurred while
   [0] Processing  Event run: 1 lumi: 133 event: 37796 stream: 2
   [1] Running path 'pDoublePuppiTau52_52'
   [2] Calling method for module L1GTDoubleObjectCond/'DoublePuppiTau5252'
Exception Message:
A std::exception was thrown.
Object doesn't have iso
----- End Fatal Exception -----------

I took the current IB (from 1h ago, latest commit 8fd467b060c, Merge pull request #1227 from gkaratha/fix_ht)

Could you have a look into this?

p.s. I guess we would have caught this with the tests for the central CMSSW PR...

@artlbv
Copy link
Author

artlbv commented Apr 2, 2024

also fyi @EmyrClement

@HaarigerHarald
Copy link

Hi Artur this is to be expected see: #1225 as we agreed internally to rename isolation to qualityScore in the case of NN taus. I'll rebase our branch and fix it in https://github.com/p2l1-gtEmulator/cmssw/tree/phase2-l1t-integration-14_0_0_pre3.

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

No branches or pull requests

2 participants