-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
tensorflow_probability.python.math conflicts with built-in math library. #179
Comments
I am using Ubuntu 17.10, Python 3.6.4 and GCC 7.2.0, Bazel 0.17.2 and bleeding edge of this repo. |
Hi @kdexd, can you share the output of |
|
Hi @kdexd, I have been unable to reproduce this. In each case, I'm running in a freshly created virtualenv with tf-nightly and other test dependencies (scipy, hypothesis, matplotlib) installed. I have tested with pythons 2.7, 3.5, and 3.6. In all cases, all tests pass. Is there something else in your setup that might be different? |
Wow, it just worked for me. I don't exactly know what was wrong before. Thanks! |
I tried to run the tests with this command:
They fail, at
//tensorflow_probability/python:monte_carlo_test
and the reason is existence ofmath
module in that directory.Full traceback
More Proof
I added these lines in
tensorflow_probability/python/math/__init__.py
:And on executing the same command, the tests fail and I get:
This definitely looks like a name conflict.
References
Exact same error here: https://ubuntuforums.org/showthread.php?t=897043
The text was updated successfully, but these errors were encountered: