-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Fixed :meth:.ThreeDCamera.remove_fixed_orientation_mobjects
#2487
Fixed :meth:.ThreeDCamera.remove_fixed_orientation_mobjects
#2487
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thank you for your PR! Can you add a test for this change? (test that the method removes a fixed orientation mobject)
:meth:.ThreeDCamera.remove_fixed_orientation_mobjects
.ThreeDCamera.remove_fixed_orientation_mobjects
I have added a test. |
for more information, see https://pre-commit.ci
Sorry forgot to get back on this issue. Thanks @behackl! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
self.fixed_orientation_mobjects
is a dictionnary, not a set as it is forself.fixed_in_frame_mobjects
.Overview: What does this pull request change?
Motivation and Explanation: Why and how do your changes improve the library?
Links to added or changed documentation pages
Further Information and Comments
Reviewer Checklist