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

Broken backward compatibility with older releases #7

Open
bfonta opened this issue Oct 16, 2024 · 2 comments
Open

Broken backward compatibility with older releases #7

bfonta opened this issue Oct 16, 2024 · 2 comments

Comments

@bfonta
Copy link

bfonta commented Oct 16, 2024

I'm using this repository within release CMSSW_10_6_29 for the resonant $HH\rightarrow bb\tau\tau$ analysis.
This commit (9cfa01925ae3e9bed808a08519aa37a3c5a68ac5) @Prasant1993 introduces references to Run 3 files that do not exist for older releases, breaking backward compatibility.
For your reference, the first error I get is (run within crab):

ERROR: ImportError: No module named mvaElectronID_RunIIIWinter22_iso_V1_cff
@Prasant1993
Copy link
Contributor

Hi @bfonta, the master branch was updated to add the Run3 IDs and energy scale+smearing corrections to run for electron and photon collections in MiniAOD. The Run3 IDs are not available anymore in Run2 if you are processing in CMSSW_10. Unfortunately backward compatibility fails.

You may try to run this postrecotools file instead for CMSSW_10 to work in Run2 : https://github.com/jainshilpi/EgammaPostRecoTools/tree/master

If you compare the two postreco files, the only difference you will get is adding Run3 IDs and corrections.

@bfonta
Copy link
Author

bfonta commented Oct 16, 2024

Thanks for the reply.
I was wondering whether we could add some conditional statements using the cmsswVersion variable (link), or else tag a particular commit to make it clear it refers to Run2.
Anyways, I've simply removed the Run3-related lines in the commit I mentioned above, and the code worked. I suppose this is equivalent to your suggested solution, but let me know otherwise.

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