-
Notifications
You must be signed in to change notification settings - Fork 10
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
Kill JointTyped #291
Kill JointTyped #291
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more iteration. Let’s discuss #290 on Monday
I think we can merge, @danbarla will have to merge in master then... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still not targeting master and that seems to create problems with extra files...
b648a46
to
7f66e4a
Compare
7f66e4a
to
3d0930a
Compare
@dellaert Sorry I totally forgot to re-target to master, and then I royally messed up and accidentally force-pushed. But after messing up once, I learned so next time I'll know how to do it right. But everything should be addressed now! |
This abandons future plans for Ball/Spherical/U-joints. Reasons:
Frank’s response (after much deliberation):
Can re-target to master if #290 is rejected. I forget why I based this branch on the other one - probably just forgot to checkout back to master before starting this work.