Fixed :meth:BarChart.change_bar_values not updating when height is 0

[hydrobeam]

Overview: What does this pull request change?

Closes #2636

• Made it so that if the previous value of a bar chart is zero during change_bar_values is zero (caught during a try/except), then a new bar is created and replaces its position in the self.bars VGroup
• Added self.insert methods to Mobject and OpenGLMobject to accomodate this.
• Refactored out bar creation into BarChart._create_bar for use in BarChart._add_bars and change_bar_values
• Added self._update_colors since if new bars are created, then their colour won't match the rest of the bar chart, so the colours will be re-initalized.

class Foo(Scene):
    def construct(self):
        values=[0 for _ in range(11)]

        chart = BarChart(
            values,
            y_range=[0, 10, 1],
            y_axis_config={"font_size": 24},
        )
        new_values=[1 for _ in range(11)]
        chart.change_bar_values(new_values)
        self.add(chart)

Before:

After:

Motivation and Explanation: Why and how do your changes improve the library?

Links to added or changed documentation pages

Further Information and Comments

Currently missing proper tests. Also, there might be value in having a singular change_bar_value.

Reviewer Checklist

• The PR title is descriptive enough for the changelog, and the PR is labeled correctly
• If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
• If applicable: newly added functions and classes are tested

[ad-chaos]

Cool! Thanks for the fix!
Left a suggestion and a potential design change and whether you think that would be a good idea.

[MrDiver]

from manim import *

class Foo(Scene):
    def construct(self):
        values=[0 for _ in range(11)]+[1,1,1]

        chart = BarChart(
            values,
            y_range=[0, 10, 1],
            y_axis_config={"font_size": 24},
        )
        new_values=[i for i in range(11)] + [-2,-3,-4]
        self.add(chart)
        self.play(chart.animate.change_bar_values(new_values))

Just a different flavor of test.

[hydrobeam]

Implemented most of the suggested changes, with some minor type/doc adjustments and a deprecation.

I also deprecated passing in a string to bar_colors (unfortunately rather manually, since our deprecated module doesn't support this type of deprecation {tried quite a bit to make deprecated_params work})

[MrDiver]

Please review requested changes!

Everything adressed but github doesn't check correctly