'size' has incorrect type (expected int, got float) about manimml HOT 13 OPEN

I couldn't reproduce the issue, but I coded up what should hopefully be a workaround.
Try updating to the local branch by running

pip install git+https://github.com/helblazer811/ManimML.git

from manimml.

It got further, but ended up with the same exception.

Constructing layers
Current layer: Convolutional2DLayer
Current layer: Convolutional2DLayer
Current layer: Convolutional2DLayer
Current layer: FeedForwardLayer
Current layer: FeedForwardLayer

Now it is running into the same issue when it tries to construct NeuralNetwork at line 17 in the example.

from manimml.

BTW, this seems like it would great to use in my ML course at Berkeley. The students definitely struggle with visualizing this stuff.

from manimml.

Could you try putting print(DEFAULT_FONT_SIZE) at the top of the script after the manim import. I'm wondering if it is not divisible by 3 and that is causing issues. I got 48 on my system.

from manimml.

from manimml.

Hmmm. Quite strange. The stack trace has the same error on line 10 of parent_layers.py?

I changed the division to integer division to see if that changes things.

Try installing again pip install git+https://github.com/helblazer811/ManimML.git

Also, integrating this into a course sounds amazing. Let me know if there is a way that I can help with that. I plan multiple extensions of this project in the future.

from manimml.

from manimml.

I think perhaps your system is using this manimpango dependency to render text instead of Latex, which is what my system is doing (I also have an M1 Macbook Pro).

Do you know what version of manimpango you have?

pip list | grep ManimPango

from manimml.

from manimml.

Perhaps try running pip install -U manimpango

from manimml.

I think it may be the ManimPango does not allow for non-integer sizes because of how it is coded under the hood. And the reason my code works is because it may not be using this dependency. Let me see if I can just ensure for the code you are running that the text font sizes are integer.

from manimml.

I think this may be related.
ManimCommunity/ManimPango#106

from manimml.

I also had the same issue with manimpango. I could not make it work even with downgrading manimpango (It caused a more serious issue).
For now I just tried to get away with casting both size and line_spacing.

from manimml.