diff --git a/animation/simple_animations.py b/animation/simple_animations.py index 54ab069d..60714406 100644 --- a/animation/simple_animations.py +++ b/animation/simple_animations.py @@ -439,10 +439,10 @@ class Succession(Animation): Animation.__init__(self, self.mobject, run_time = run_time, **kwargs) - # Beware: This does NOT take care of updating the subanimation to 0 + # Beware: This does NOT take care of calling update(0) on the subanimation. # This was important to avoid a pernicious possibility in which subanimations were called # with update(0) twice, which could in turn call a sub-Succession with update(0) four times, - # continuing exponentially + # continuing exponentially. def jump_to_start_of_anim(self, index): if index != self.current_anim_index: self.mobject.remove(*self.mobject.submobjects) # Should probably have a cleaner "remove_all" method... diff --git a/camera/camera.py b/camera/camera.py index 17b2d744..a9a27021 100644 --- a/camera/camera.py +++ b/camera/camera.py @@ -152,6 +152,9 @@ class Camera(object): ) mobjects = list_difference_update(mobjects, all_excluded) + # Should perhaps think about what happens here when include_submobjects is False, + # (for now, the onus is then on the caller to ensure this is handled correctly by + # passing us an appropriately pre-flattened list of mobjects if need be) return sorted(mobjects, lambda a, b: cmp(z_buff_func(a), z_buff_func(b))) def capture_mobject(self, mobject, **kwargs):