-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Scene Caching Tries to Access the Wrong File in Certain Circumstances #451
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
Comments
Hi, thanks for the report. Did you try to run this with |
Yeah, I did, and it works fine when using that flag. |
Well, I just found somewhere else it breaks without rotation. Let me try to isolate what's breaking it. |
Here's a reduced version of another case I found: class Issue3DCache2(Scene):
def construct(self):
self.wait()
m = Square()
self.add(m)
#self.wait() Run it the first time without the commented wait, and the second time with it. Then a similar problem occurs. However, unlike before, the wait has to be added between runs. |
Alright, thanks so much for that. |
This was the second one (without the rotation), even though the name is still |
I was wanting to jump to an earlier commit so I wouldn't have to deal with this problem any more, and I found that the bug was introduced in commit 1a516db. |
Maybe this is fixed by #489 . I tested it, and it worked. Could you confirm ? |
If it worked for you, I'm fine trusting that. I don't really know any way to test it right now short of uninstalling my copy and then installing from your fork, and that seems like more trouble than it's worth. |
Sorry for that. nevertheless, now everything is good as #489 is merged :D |
Consider the following scene:
when I run manim once, it makes a black screen as expected, but when I run it again, it crashes saying that it can't find the partial movie file.
When running
manim test.py Issue3DCache -pl
the first time:When running it again:
The text was updated successfully, but these errors were encountered: