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.

Sign up for GitHub

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

Jump to bottom

Spline orbital optimization tracker #4865

Open
4 of 10 tasks
camelto2 opened this issue Dec 8, 2023 · 2 comments
Open
4 of 10 tasks

Spline orbital optimization tracker #4865

camelto2 opened this issue Dec 8, 2023 · 2 comments

Comments

@camelto2
Copy link
Contributor

camelto2 commented Dec 8, 2023 •
edited by jptowns
Loading

Issue to track all the variations of splines and whether or not they support orbital rotation

Normal Splines

Hybrid Representation
@prckent prckent changed the title Spline applyRotation tracker Spline orbital optimization tracker Dec 8, 2023
@jptowns
Copy link
Contributor

jptowns commented Dec 21, 2023

I have an idea: Let's get rid of C2R entirely. It's an edge case. And, while I don't know for certain, i suspect it is very rarely used in production. Perhaps we can try deprecating it and see if anyone complains. If, in 6 months, say, no one has noticed we can just remove entirely. Thoughts @ye-luo ? @prckent ?

@prckent
Copy link
Contributor

prckent commented Dec 23, 2023 •
edited
Loading

This is a good suggestion. The question I have is how often the c2r path is used in practice (vs in theory )? Something to discuss in the new year.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@prckent @camelto2 @jptowns