Closed jeertmans closed 1 month ago
Attention: Patch coverage is 54.54545%
with 15 lines
in your changes missing coverage. Please review.
Project coverage is 79.19%. Comparing base (
1dbd2fd
) to head (c817781
). Report is 66 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
manim_slides/present/player.py | 54.54% | 15 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Hey jeertmans, you might not remember me but I spoke with you on discord about a month ago about this issue. I know it's been a while but I was just wondering if there was any update as to how to get this working, if I should change pyside6 versions, change over to pyqt6, or install a different version of manim-slides entirely. If there is nothing and I do end up having to change my version of manim-slides, can you let me know what version to install and which qt bindings to install with it? Thanks
Hey jeertmans, you might not remember me but I spoke with you on discord about a month ago about this issue. I know it's been a while but I was just wondering if there was any update as to how to get this working, if I should change pyside6 versions, change over to pyqt6, or install a different version of manim-slides entirely. If there is nothing and I do end up having to change my version of manim-slides, can you let me know what version to install and which qt bindings to install with it? Thanks
Hi @pianoboy71! Unfortunately, I did not try anything further, as I am quite busy with my research work, and I am not an active Windows user :-/
Maybe installing it through WSL on Windows might be a "fix", but I hope to come with a better fix one day. If anyone is willing to put some time to help on this issue, please do so!
Perhaps of little value, but I've noticed that using --hide-info-window
prevents the presentation from crashing and thus allows the user to play manim-slides presentations on windows systems.
Perhaps of little value, but I've noticed that using
--hide-info-window
prevents the presentation from crashing and thus allows the user to play manim-slides presentations on windows systems.
Indeed, the fact that the info window duplicate the video stream seems to crash the app, and a solution seems to be to hide it from the start. I'd love to provide a better fix, but thank you for mentioning it :-)
Let me know if you've come across any other fixes as I am using your excellent library on Windows right now. I'd love to contribute if I ever find the time to do so :)
Let me know if you've come across any other fixes as I am using your excellent library on Windows right now. I'd love to contribute if I ever find the time to do so :)
Well, if you have some time, I'd love to get your help on Windows (see my message on Reddit)!
I think that, features-wise, Manim Slides is quite stable (at least the manim-slides present
command). I'd like to prepare a v6
that is: (1) easier to install and (2) works on most platforms by default.
v6 will need (1) the Qt issue to be fixed (https://bugreports.qt.io/browse/QTBUG-118501) and also fixing any issue regarding incompatible Qt / Python version / OS triplets.
If you feel confident, I suggest you fork and clone this repo, checkout this branch, and start modifying manim_slides/present/player.py
and see if that fixes the issues.
Another neat thing would be to create tests that capture those failing tasks, so we can actually test future fixes! Please let me know if you ever need help!
Closing this PR (and related issue) as it does not seem to be needed anymore, as of #465.
Apparently, the "recommended* versions of Qt (i.e., Qt 6.5.1 and 6.5.2) produces errors.
I could not exactly re-produce #315, but I also have
failed to get textures for frame; format: 172 textureConverter null
in the output, as https://github.com/jeertmans/manim-slides/issues/315#issuecomment-1965038997 mentioned.After searching a bit, this feels like the faulty lines are (at least related to) https://github.com/jeertmans/manim-slides/blob/1dbd2fdde58030867f15d7bf0ecede11d59d6903/manim_slides/present/player.py#L245-L247.
I could not yet find a fix for that, but I created this PR for testing purposes... I also added quite a few debug prints, for the future.
When upgrading Qt to 6.5.3 and above, the error message disappear, but we then have the same visual issue as described in #293.
This is the logs when I comment the lines mentioned:
If I put the lines back, the message is:
Now with
PySide6==6.5.3
: