-
Notifications
You must be signed in to change notification settings - Fork 127
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
Comments
I guess that working point is not available anymore (on R22). Which seems to be confirmed by looking at this: From that page, it seems only these WPs are supported in R22: JETIDNONE 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? |
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;
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. |
Great! Yes, that sounds awesome. Thanks a lot! |
Still need to test it but have a draft for this now; #1522 |
Closing issue since PR with fixes was merged. |
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;
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 usingJETIDBDTLOOSENOTTIGHT
, so I don't understand why that works.Any wisdom you can offer is gratefully received.
The text was updated successfully, but these errors were encountered: