jeertmans / manim-slides

Tool for live presentations using manim
https://manim-slides.eertmans.be
MIT License
446 stars 48 forks source link

[BUG] manim-slides present not pausing properly between animation #364

Closed mmcilree closed 8 months ago

mmcilree commented 8 months ago

Description

Just updated to the latest version and the manim-slides present doesn't seem to be working correctly:

It will play each animation on self.next_slide() as expected but then everything disappears until the next keypress. This happens with any of the examples I've tried, including BasicExample where I see the GrowFromCenter animation, but then the display goes black until I press to continue.

Version

manim-slides, version 5.1.0 Manim Community v0.18.0

Platform

macOS Monterey on an M1 Pro MacBook

Screenshots

No response

Additional information

No response

jeertmans commented 8 months ago

Hello, could you provide the source code of your slides?

Also, if you could include a small video recording, that would help! Thanks!

mmcilree commented 8 months ago

Had a look at the other issues and haved fixed it by downgrading PySide6 to version 6.5.0.

pip install pyside6==6.5.0 

I guess it's related to #315 and #293

jeertmans commented 8 months ago

Aah yeah, nevermind this is probably because you use a Qt version that has this issue :/ You have a flashing black screen at the end of each video?

jeertmans commented 8 months ago

Yes indeed @mmcilree, you replied faster than me 😅 I thought that I restricted the PySide6 version correctly.. how did you install PySide6?

mmcilree commented 8 months ago

I think what happened is I had manim-slides installed previously (via pip, a few months ago), hadn't updated it until today, then did

pip3 install -U manim-slides

which maybe kept the version of PySide6 I had already?

jeertmans commented 8 months ago

Actually, as of v5.1, pip install manim-slides does not install PySide6 anymore, so this explains why the version constraint was not applied. To install PySide6, you now need pip install manim-slides[pyside6].

This change was mainly motivated by the fact that correct PySide6 bindings are not available on all platforms and Python versions, and I didn't want to limit people from installing Manim Slides, when they only every only the manim-slides convert features.

I am closing this issue as this is indeed a duplicate of #293, and I already filed an issue to Qt: https://bugreports.qt.io/browse/QTBUG-118501.

Because this is a Qt bug, it is present with both PySide6 and PyQt6 bindings :-/