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

Error 'JETIDBDTLOOSENOTTIGHT' is not a member of 'TauAnalysisTools' #1521

Closed
HenryDayHall opened this issue Feb 7, 2022 · 5 comments
Closed

Comments

@HenryDayHall
Copy link
Contributor

Apologies in advance for being pretty naive about a lot of things, I'm hoping that you can give me a few pointers in the right direction. I'm supposed to be updating some code to work with a more recent version of athena, and this code also requires xAODAnaHelpers. Currently, when I try to compile I get the error;

Error 'JETIDBDTLOOSENOTTIGHT' is not a member of 'TauAnalysisTools'

and indeed, JETIDBDTLOOSENOTTIGHT was removed from recent versions of athena, in this commit (only a month ago). But then some of the issues posted here seem to imply that athena is not needed for this package, like this one. So I'm not sure I have properly understood the relationship between this package and athena. Perhaps there should be a way to update athena and not hit this issue?

Interestingly, I can make xAODAnaHelpers on it's own with the same version of athena that breaks it in conjunction with my code. It doesn't seem to matter that xAODAnaHelpers is referencing JETIDBDTLOOSENOTTIGHT and that doesn't exist anymore until I combine xAODHelpers with additional code. But this file is certainly using JETIDBDTLOOSENOTTIGHT, so I don't understand why that works.

Any wisdom you can offer is gratefully received.

@jbossios
Copy link
Contributor

jbossios commented Feb 7, 2022

I guess that working point is not available anymore (on R22). Which seems to be confirmed by looking at this:
https://gitlab.cern.ch/atlas/athena/-/blob/master/PhysicsAnalysis/TauID/TauAnalysisTools/doc/README-TauSelectionTool.rst

From that page, it seems only these WPs are supported in R22:

JETIDNONE
JETIDRNNVERYLOOSE
JETIDRNNLOOSE
JETIDRNNMEDIUM
JETIDRNNTIGHT

Most likely, no one yet tested xAH on R22 with taus. Would you mind removing all non-supported WPs in the file below and test again?

https://github.com/UCATLAS/xAODAnaHelpers/blob/r22/master/Root/TauSelector.cxx

If that solves it, could you please make a PR to fix this?

@HenryDayHall
Copy link
Contributor Author

HenryDayHall commented Feb 7, 2022

Many thanks, making that change does indeed get rid of this error.

I now get a few more errors that probably run in the same vein;

error: 'SFEleOLRHadTau' is not a member of 'TauAnalysisTools'                                                        

So would it be best if I try to fix anything that looks related and then make a PR with all the related updates? I have a good idea how to go about that now.

@jbossios
Copy link
Contributor

jbossios commented Feb 7, 2022

Great! Yes, that sounds awesome. Thanks a lot!

@HenryDayHall
Copy link
Contributor Author

Still need to test it but have a draft for this now; #1522

@jbossios
Copy link
Contributor

jbossios commented Mar 4, 2022

Closing issue since PR with fixes was merged.

@jbossios jbossios closed this as completed Mar 4, 2022
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