-
Notifications
You must be signed in to change notification settings - Fork 483
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
ImportError: cannot import name 'imread' from 'scipy.misc' #59
Comments
Download the scripts again. You have an older version. The new one has scipy imports commented out. |
@titu1994 I downloaded latest versions with conda. tensorflow-gpu 1.13.1 |
This issue was patched recently. Update the codebase, not the libraries. |
@titu1994 Please don't close issues before you get a successful response from the topic starter. That's not a professional way to resolve problems. If you close an issue it doesn't mean that it's resolved.
Everything is installed git-pulled and there's a problem in your code again. Running as this:
|
@titu1994 Also: Always specify the exact version of all dependencies that the application is stable with. Every good developer knows that updating the version of any dependencies can cause the application to break down. |
@FlamesoFF Many points to raise here.
I closed the issue immediately, because I already knew you did not have the latest code and therefore once you did, it would work properly. Therefore this issue was closed.
There's a full guide to the scripts in the readme, a full set of applications to run them with a GUI, and online GPU access via colab. How much more should I possibly do to make it less of a problem for people to run this code? Seriously, I could not care less about resolving the bugs of this project due to library changes spanning 3+ years. If you peer into my github sometime, you'll note I have at least 90+ projects that I have to manage, all on my own time. The fact that I still keep this one updated, is solely due to the fact that it is enjoyable once in a while. Nothing else. If you feel that this is "Such nonsense", by all means, use some other library for your needs. Lastly, end rant, and the utils script has been patched to correct the issue you had. |
@titu1994 I understand that you're full-time developer on your main job and this is your passionate project. But even full-time job is not a excuse for not providing dependencies version in requirements.txt.
Okay, I will open another if it will still has place after update.
I'm not expect
It's not a complaining. It's a question. Otherwise how I'm suppose to know what exact versions should be used for stable app work?
Sorry for that. No offense here, man. You're project is good, but I only wanted to give couple of tips about how to document projects for people. |
@titu1994 Well, after installing requirements with |
py Network.py "./images/inputs/1.jpg" "./images/inputs/style.jpg" C:\Users\flamesoff\AppData\Local\Programs\Python\Python37\lib\site-packages\numpy\_distributor_init.py:32: UserWarning: loaded more than 1 DLL from .libs: C:\Users\flamesoff\AppData\Local\Programs\Python\Python37\lib\site-packages\numpy\.libs\libopenblas.IPBC74C7KURV7CB2PKT5Z5FNR3SIBV4J.gfortran-win_amd64.dll C:\Users\flamesoff\AppData\Local\Programs\Python\Python37\lib\site-packages\numpy\.libs\libopenblas.TXA6YQSD3GCQQC22GEQ54J2UDCXDXHWN.gfortran-win_amd64.dll stacklevel=1) Traceback (most recent call last): File "Network.py", line 5, in <module> from scipy.misc import imread, imresize, imsave, fromimage, toimage ImportError: cannot import name 'imread' from 'scipy.misc' (C:\Users\flamesoff\AppData\Local\Programs\Python\Python37\lib\site-packages\scipy\misc\__init__.py)
Everything is installed in accordance with requirements from
readme.md
.The text was updated successfully, but these errors were encountered: