Skip to content
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

Convert t + q to t + t in more cases #1888

Closed
KTibow opened this issue Dec 17, 2023 · 0 comments · Fixed by #1889
Closed

Convert t + q to t + t in more cases #1888

KTibow opened this issue Dec 17, 2023 · 0 comments · Fixed by #1889

Comments

@KTibow
Copy link
Contributor

KTibow commented Dec 17, 2023

Is your feature request related to a problem? Please describe.
Right now the "t + q -> t + t" conversion only converts it when there's two curves with the same delta. It doesn't account that t can be used in more cases, like where it reflects properly.

Describe the solution you'd like
Keep track of the quadratic control point somehow, for each q testing if the predicted control point matches with the actual control point, and replacing it with t if so.

Describe alternatives you've considered
Implement it myself? Not sure what to say

Additional context
For many circles, this would 66% the size (more realistically like 75% but still this is a big deal)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant