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
BUG : fix #3805 #3808
BUG : fix #3805 #3808
Conversation
Enusure that norm.v{min,max} are not `None` before expanding them to ensure that 0 < vmax-vmin.
You beat me to it! That's the fix; does it need a test? |
writing it right now, |
Looks good to me apart from the pep8 issue on Travis in the test. (I am not sure that pep8 on the tests makes that much sense but I guess it is easy to fix in this case) |
512b4a8
to
661db3f
Compare
fixed + force pushed. Sorry had my editor configured wrong. |
Should this be backported? |
back-ported as 7ba2c2a Good catch! I actually made the change against 1.4.2, but targeted the PR against the wrong branch..... |
Enusure that norm.v{min,max} are not
None
before expandingthem to ensure that 0 < vmax-vmin.