Repository: ManimCommunity/manim
File: manim/animation/animation.py (Python)

Code before:
from copy import deepcopy
from typing import TYPE_CHECKING, Callable, Iterable, Optional, Tuple, Type, Union

if TYPE_CHECKING:
    from manim.scene.scene import Scene

from .. import logger
from ..mobject import mobject, opengl_mobject
from ..mobject.mobject import Mobject
from ..mobject.opengl_mobject import OpenGLMobject
from ..utils.rate_functions import smooth

DEFAULT_ANIMATION_RUN_TIME: float = 1.0
DEFAULT_ANIMATION_LAG_RATIO: float = 0.0


class Animation:
    def __new__(
        cls,
        mobject: Optional[Mobject] = None,
        *args,
        use_override: bool = True,
        **kwargs,
    ):
        if isinstance(mobject, Mobject) and use_override:
            func = mobject.animation_override_for(cls)
            if func is not None:
                anim = func(mobject, *args, **kwargs)
                logger.debug(
                    (
                        f"The {cls.__name__} animation has been is overridden for "
                        f"{type(mobject).__name__} mobjects. use_override = False can "
                        f" be used as keyword argument to prevent animation overriding."
                    )
                )
                return anim
        return super().__new__(cls)

    def __init__(
        self,
        mobject: Optional[Mobject],
        # If lag_ratio is 0, the animation is applied to all submobjects
        # at the same time
        # If 1, it is applied to each successively.
        # If 0 < lag_ratio < 1, its applied to each
        # with lagged start times
        lag_ratio: float = DEFAULT_ANIMATION_LAG_RATIO,
        run_time: float = DEFAULT_ANIMATION_RUN_TIME,
        rate_func: Callable[[float], float] = smooth,
        name: str = None,
        remover: bool = False,  # remove a mobject from the screen?

Review comment (suggestion):
Nonlblocking but changing the check to
```suggestion
        if mobject is not None and use_override:
```
makes more sense to me

Code after:
from copy import deepcopy
from typing import TYPE_CHECKING, Callable, Iterable, Optional, Tuple, Type, Union

if TYPE_CHECKING:
    from manim.scene.scene import Scene

from .. import logger
from ..mobject import mobject, opengl_mobject
from ..mobject.mobject import Mobject
from ..mobject.opengl_mobject import OpenGLMobject
from ..utils.deprecation import deprecated
from ..utils.rate_functions import smooth

DEFAULT_ANIMATION_RUN_TIME: float = 1.0
DEFAULT_ANIMATION_LAG_RATIO: float = 0.0


class Animation:
    def __new__(
        cls,
        mobject: Optional[Mobject] = None,
        *args,
        use_override: bool = True,
        **kwargs,
    ):
        if isinstance(mobject, Mobject) and use_override:
            func = mobject.animation_override_for(cls)
            if func is not None:
                anim = func(mobject, *args, **kwargs)
                logger.debug(
                    (
                        f"The {cls.__name__} animation has been is overridden for "
                        f"{type(mobject).__name__} mobjects. use_override = False can "
                        f" be used as keyword argument to prevent animation overriding."
                    )
                )
                return anim
        return super().__new__(cls)

    def __init__(
        self,
        mobject: Optional[Mobject],
        # If lag_ratio is 0, the animation is applied to all submobjects
        # at the same time
        # If 1, it is applied to each successively.
        # If 0 < lag_ratio < 1, its applied to each
        # with lagged start times
        lag_ratio: float = DEFAULT_ANIMATION_LAG_RATIO,
        run_time: float = DEFAULT_ANIMATION_RUN_TIME,
        rate_func: Callable[[float], float] = smooth,
        name: str = None,