-
-
Notifications
You must be signed in to change notification settings - Fork 148
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
Installing under Anaconda #25
Comments
You ran |
It seemed to install OK. But when I try to run one of the examples, I get this error ImportError Traceback (most recent call last) ImportError: cannot import name 'Convolutional2DLayer' from 'manim_ml.neural_network' (/usr/local/anaconda3/envs/my-manim-environment/lib/python3.8/site-packages/manim_ml/neural_network/init.py) |
Wonder if it is a path issue? |
Hmmm. I tried to simplify imports by having everything importable from Can you try and do Does that fix the issue? |
Also can you specify which version of manim_ml you have installed? You can do this by running |
I made the change and was able to import |
Name: manim-ml |
I should mention that I am trying all these inside a Jupyter notebook |
Awesome. I'll look into the importing issue. Thanks for the help! |
I have not tested this at all in Jupyter notebook environments. So it may not work there. However, it looks like the core ManimCommunity library did work to support Jupyter. https://docs.manim.community/en/stable/installation/jupyter.html |
Yes, I am able to run all the standard examples for ManimCommunity from inside Jupyter notebooks. Thank you, let me know if you want me to do any further tests. |
Amazing! Feel free to go through more of the examples in the |
OK, I sort of got it working in Jupyter. I think something is broken with a path (at least on my installation) but the two things I did that got it work are:
The second step tells me that somehow the pip installation probably did not properly set the path. |
In the code, what exactly did you change? I am not sure if you issue is specific to jupyter. I wonder if you could copy the example code at the top of the Readme into a file called |
This is the cell that runs successful (after loading manually the lib in the previous cell)
|
So the name of the object is What did you do to manually load the library in the previous cell? Again, thanks so much for all of the help! |
Just run this command. It loads the file and then I execute the cell.
|
Ok. I am unsure if this is a jupyter-specific issue. I will try testing it out in a jupyter notebook. |
Great, thank you! |
I think I just forgot to add the |
It solved half of the problem in the sense that now I can use
to import the layers. However, I am still not able to avoid having to load manually the library, so the path issue continues. |
Hmmm. Is it possible that you are not running the Jupyter notebook with the conda environment's kernel? |
I take it back. I restarted the kernel and go it to work like this First cell:
Second cell:
|
By the way, I have a suggestion for future additions. It would be nice if it could include additional type of layers as used in Keras, such as Max_Pooling2D, Concatenate and Up_Sampling. That would be super! |
Amazing! |
Great ideas. I already have implemented Max_Pooling2D. You can check it out in one of the readme examples. |
Oh, that's becoming really nice! |
By the way, is padding something to be accounted for? |
That’s another thing I’m planning to add soon. The feature I’m really
excited about is haven’t an effect where the camera zooms in on the active
layers and moves forward with the forward pass. That way I can visualize
super deep architectures.
…On Mon, Jan 30, 2023 at 2:17 PM sck-at-ucy ***@***.***> wrote:
By the way, is padding something to be accounted for?
—
Reply to this email directly, view it on GitHub
<#25 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADMGLRSYNTYMUGWCX2PRRHLWVAHUZANCNFSM6AAAAAAUKMGASM>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Perfect |
I added padding. For some reason I can't upload the video, but you can see it here: https://twitter.com/alec_helbling/status/1620641566024171520?s=20. |
It would be amazing if eventually you get it to read in the model summary files generated by Keras and then do the animation. Should’t be that hard as you already have most of the hard work.
|
I have been unable to get Manim_ml to properly install under Anaconda distribution. Any advise?
The text was updated successfully, but these errors were encountered: