-
Notifications
You must be signed in to change notification settings - Fork 1
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
Cannot import package due to sklearn dependency #10
Comments
Hi @kellieotto , this bug has been fixed. Please let me know if other problems. Thanks. |
@Ji-Zhang That error seems to be fixed, great! I'm still running into import issues.
Seems to be coming from tensorflow. |
Hi @kellieotto, may I ask in which environment you are using the TensorFlow? There is not much I can do in the package to fix this. If you are using TensorFlow with GPU, you need to install CUDA and cuDNN. Please follow instructions on https://www.tensorflow.org/install/ If you have already install CUDA and cuDNN, but still get this error, then you probably forgot to export your libraries: for Linux, you may need to set LD_LIBRARY_PATH to include CUDA libraries. If the above can not fix the problem, please let me know so I can further help. Thanks. |
Hi @Ji-Zhang, sorry for the huge time delay between responses on this. I have installed everything now. I am running the code example you have in the README and get this error:
I think it's related to package versions + pickling. I found this issue that seems related. |
Hi @kellieotto , this bug should be fixed now. Could you please test it again? Thanks in advance. |
Sorry @Ji-Zhang it's still not working when I run I did ran |
Hi @kellieotto , sorry for the inconvenience. I changed the way to load the trained model. Could you please try it again? |
This issue is part of your JOSS review.
I was able to install the package but can't import it. I get the following error
seems related to this? https://stackoverflow.com/questions/59439096/importerror-cannnot-import-name-imputer-from-sklearn-preprocessing
The text was updated successfully, but these errors were encountered: