Skip to content

LGTM.com - false positive: if/elif/else comparison incorrect (unreachable, always true) #1989

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

Open
kratsg opened this issue Sep 20, 2019 · 1 comment

Comments

@kratsg
Copy link

kratsg commented Sep 20, 2019

Description of the false positive

alpha is a variable provided through the function and takes on any value from (-inf,+inf).

https://lgtm.com/projects/g/diana-hep/pyhf/snapshot/4a8bb79dcb5a18156090441c789404248a4872bb/files/src/pyhf/interpolators/code2.py?sort=name&dir=ASC&mode=heatmap#x490ee0f59f7109a3:1

(and related)

https://lgtm.com/projects/g/diana-hep/pyhf/snapshot/4a8bb79dcb5a18156090441c789404248a4872bb/files/src/pyhf/interpolators/code2.py?sort=name&dir=ASC&mode=heatmap#x4162964d9ead9482:1

@kratsg kratsg changed the title LGTM.com - false positive LGTM.com - false positive: if/elif/else comparison incorrect (unreachable, always true) Sep 20, 2019
@matthewfeickert
Copy link

If I change it to elif alpha >= -1 then LGTM is satisfied, but the logic is clearly the same as -1 <= alpha <= 1 in our case. So we're confused why LGTM is flagging this.

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

No branches or pull requests

5 participants