-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[Solved] Installing mlagents doesn't generate a compatible Tensorflow. #4033
Comments
I have also tried with Anaconda.
|
@jdunwoody This appears to be a general tensorflow problem, and not specific to ML-Agents. Have you tried this? tensorflow/tensorflow#22794 (comment) (which I found by following the link to https://www.tensorflow.org/install/errors) |
That was exactly it! Thankyou very much! For this problem
Install this I also looked into the Tensorflow messageboards for reasons for this error but didn't get to the bottom of the discussion - to find this gem! Too many suggestions from the Tensorflow community suggesting various combinations of Would it be worthwhile adding this dependency to the mlagents installation docs, given that this requirement is now default for Linux and Windows Tensforflow? I can raise a pull request if you'd like me to do it. |
Glad that fixed it! I think we should definitely add it to the FAQ here: https://github.com/Unity-Technologies/ml-agents/blob/master/docs/FAQ.md#tensorflow-dependency An additional option would be to do a try-catch around the tensorflow import here
and log a better error message if the exception contains "DLL load failed". I like this option since it keeps the solution close to the problem, unfortunately I'm not on Windows so it's hard for me to test. As for the installation instructions, @mmattar (who's currently on PTO until the end of the week) has some opinions on what should and shouldn't go in the install guide, and recently rewrote some large swaths of it. If you want to make a pull request for either the FAQ or the try-catch, please do. You're welcome to for the installation instructions too, it just might take a few days to get it reviewed. |
I'll submit a PR for at least the FAQ. I'll see how the guard code could work. |
* Added Microsoft cpp dependency for windows users in faq (#4033) * Update docs/FAQ.md Co-authored-by: Chris Elion <celion@gmail.com> * Update docs/FAQ.md Co-authored-by: Chris Elion <celion@gmail.com> Co-authored-by: Chris Elion <celion@gmail.com>
Could this be made more obvious in the Installation Guide? I think it should be mentioned there pretty clearly, since I only found out the solution to this problem by searching through previous issues and finding this. I didn't even consider looking for a FAQ since I interpreted Tensorflow's problem as a problem or bug rather than a question. |
Actually my bad, repairing the C++ redistributables did not solve my problem. I still get the following:
(not sure why it says win32, I'm definitely 64-bit) EDIT: Needed to restart after CUDA install, ok good now ^_^ |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
mlagents-learn
generates a stack trace because of Tensorflow with any commanda
requirements.txt
file would be helpful to lock down the various dependencies.Tensorflow messageboards suggest downgrading
protobuff
versions andtensorflow
versions. Could someone freeze a recent version of their environment and post it? This might help find a combination of dependencies that work together with the latest release of mlagents.To Reproduce
Environment:
2018.4.23f1
Microsoft Windows 10 Home
release_2_branch
2.2.0
The text was updated successfully, but these errors were encountered: