-
Notifications
You must be signed in to change notification settings - Fork 300
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
Additional constraints #975
Conversation
adam-urbanczyk
commented
Feb 1, 2022
•
edited
Loading
edited
- simplification of the assy solver
- validation
- PointOnLine
- FixedPoint
- FixedAxis
- FixedRotation
- docs
I did not add the new constraints yet, but @marcus7070 @lorenzncode could you take a look to see if the reworked version is more readable? |
Codecov Report
@@ Coverage Diff @@
## master #975 +/- ##
==========================================
+ Coverage 96.17% 96.26% +0.08%
==========================================
Files 40 40
Lines 9292 9341 +49
Branches 1109 1103 -6
==========================================
+ Hits 8937 8992 +55
+ Misses 208 205 -3
+ Partials 147 144 -3
Continue to review full report at Codecov.
|
+initial work on FixedPoint
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.
This looks fantastic @adam-urbanczyk, thanks so much for doing it!
Co-authored-by: Marcus Boyd <[email protected]>
Co-authored-by: Lorenz <[email protected]>
Ok, I think that this is it @jmwright @lorenzncode @marcus7070 . The only thin I don't really like is the automatic fixed constraint handling. Maybe in the future (not in this PR) we should require an explicit |
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.
I've added a few minor suggestions for docs, error messages.
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.
Looks good, thanks for all the work @adam-urbanczyk
I'm relying on @marcus7070 and @lorenzncode quite a bit for this review since it seems that they understand the subtleties of the constraint system better than I do.
Co-authored-by: Lorenz <[email protected]> Co-authored-by: Jeremy Wright <[email protected]>
Great, I'll merge tomorrow. |