-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: NumberLine
number_to_point
broken when using add_tip
#3820
base: main
Are you sure you want to change the base?
Fix: NumberLine
number_to_point
broken when using add_tip
#3820
Conversation
… end positions of an object would include the length of an added tip, instead of the expected length without the tip.
…ot appear. Also, the size of a tip when added after creation with default parameters did not match the size when added during creation.
There's an issue now where the expected values in several of the tests (the ones that depend on |
Hello! Thanks for taking on this issue, and helping out with developing Manim :)
You can see how to regenerate the graphical data in our docs However, I would think carefully before you do, and look at the difference between your results and the current test data. E.g. for the test Some possible options to explore is a boolean parameter to toggle between |
I think one possible way forward is to have an input parameter that control whether the arrow tip isn't create insides or outside of the end of the line. The default value of this parameter can be the original implementation (with the tip generated inside the end of the line), but it would also allow users to configure the arrow tips to their liking. By setting the default to the tip inside, the tests (hopefully) should generate data that matches the expected values. My guess is we'd have to add this new parameter to all classes that use / are subclasses of |
Overview: What does this pull request change?
Fixed issue where
NumberLine
'snumber_to_point
method didn't convert from number line value to image position correctly after callingadd_tip
after creation. See issue #3740.Motivation and Explanation: Why and how do your changes improve the library?
NumberLine
'snumber_to_point
method now converts from number line value to image position correctly after callingadd_tip
.Links to added or changed documentation pages
No documentation changed.
Further Information and Comments
Also fixed was a small issue with the final tick mark of a number line not always appearing when a tip was added.
Reviewer Checklist