Manim: remove animation.update

[huguesdevimeux]

Bsically the title. this method is useless. See the code and the comment :
https://github.com/ManimCommunity/manim/blob/5dd7bfe644e57d428deaad18f0ab8d3772d1ad01/manim/animation/animation.py#L115-L120

[leotrs]

We should add a deprecation warning

[musabkilic]

I think adding a warning for a while to give people some time to update is a better idea, similar to:
https://github.com/ManimCommunity/manim/blob/97de34692de7442845f9b935c6d1481784a739d4/manim/mobject/svg/tex_mobject.py#L334-L349

[huguesdevimeux]

I think adding a warning for a while to give people some time to update is a better idea, similar to:

Yes (that's basically what @leotrs said lol)

[musabkilic]

I can add one with:

"animation.update(alpha) has been deprecated (due to its confusing name) "
"in favour of animation.interpolate(alpha). Please use interpolate() instead!"
let me know if it sounds good.

[leotrs]

"animation.update() has been deprecated. Please use animation.interpolate() instead."

Fewer words, same meaning.

[meanmachin3]

Can I take this up ?

[leotrs]

Yes, thanks!

[musabkilic]

Sorry, I didn't see @meanmachin3 comment. Should I sent a pull request anyway?

[meanmachin3]

@musabkilic It's okay :)
I'll work on something else. You could take it

[musabkilic]

@meanmachin3 Thanks! :)

[cobordism]

closed due to #503