egui_plot: Force use of min_auto_bounds if auto_bounds is disabled by default. Fixes #3893.

[aholtzma-am]

The logic for determining bounds was not using the values supplied by include_x()/include_y() after the initial plot call when auto_bounds() was false. This change forces the use of the manual limits when default_auto_bounds is false.

• Closes egui_plot 0.25.0 - No auto_bounds + include_X/Y does not render anything until double click egui_plot#25

[aholtzma]

Looks like this just needs a label applied.

[emilk]

This reads backwards to me now. If the user has said "DON'T apply automatic bounds" then we set the bounds to min_auto_bounds

[aholtzma-am]

The min_auto_bounds includes the bounds manually specified by include_x()/include_y(). So if you turn off auto bounds, and supply bounds via include_xy(), it won't use them except for the first call.

[Fabus1184]

Perhaps it would be better to factor this out into two methods, expand_{x,y} to expand the bounds to always show the specified value, and limit_{x,y} to limit the axis range, both applying regardless of whether auto bounds is active or not.
Of course if the user moves the plot, this should not apply. You could always disallow moving the plot if you don't want that.

[emilk]

egui_plot has recently been moved to its own repository, at https://github.com/emilk/egui_plot

This will hopefully speed up its development by having more reviewers and maintainers.

Please re-open this PR at https://github.com/emilk/egui_plot/pulls

See also:

• Looking for maintainer egui_plot#58
• Move egui_plot to its own repo #4828