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

Code duplication found #884

Closed
adavidzh opened this issue Dec 16, 2023 · 0 comments · Fixed by #1017 · May be fixed by #989
Closed

Code duplication found #884

adavidzh opened this issue Dec 16, 2023 · 0 comments · Fixed by #1017 · May be fixed by #989

Comments

@adavidzh
Copy link
Member

I just noticed this code duplication in case some refactoring effort surfaces: https://github.com/cms-analysis/HiggsAnalysis-CombinedLimit/blob/man/src/ProcessNormalization.cc#L112-L129 and https://github.com/cms-analysis/HiggsAnalysis-CombinedLimit/blob/main/src/AsymPow.cc#L33-L56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment