-
-
Notifications
You must be signed in to change notification settings - Fork 142
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
Angle Constraint Bug - $50 #261
Comments
Managed to recreate a minimal example. When trying to set the angle to for example 50 degrees, it breaks. |
Uncaught exception happens when solving the system Pointing to this particular line Compared to the upstream solvesystem repo, the one we use with the python bindings is 1100 commits behind. I think the realthunder/solvespace repo desperatly needs an update from upstream. It might be worth trying this version instead: https://github.com/KmolYuan/solvespace https://pypi.org/project/python-solvespace/ But I think @hlorus knows best about the different python ports of solvespace. |
Absolutely agree with that, we should probably reach out to realthunder.
That's actually what i used to start with but i've decided to switch to the version that is used now back because it didn't seem to be maintained iirc.
I don't really know alot about the differences. There's just no available python package of the official solvespace solver as far as i know. |
The https://github.com/KmolYuan/python-solvespace is archived, but he also has https://github.com/KmolYuan/solvespace which is maintained and a lot closer to upstream. I'm trying to use it atm, but there are quite a few differences (obvi) |
in your code shown here, how are you running the solver.py script? Do you run the solver.py from your python IDE? Sorry for the basic question, I really appreciate your input on these GitHub pages amrsoll. |
No worries :) I created a vscode file that runs a debug server and connects to the vscode debug client. It should happen before right after all the imports in the addons are done. If using VSCode / Codium you should be able to press F5 to run the selected debugger See #206 |
I git bisected this issue because it seemed to work at some point in the past. 4caa0fc#diff-116ba881a2b88b2046ed43e0ccf5660900385cdfd8bb3beea1a41a7cc6c49f83 Here is the corrosponding PR: |
The angle constraint fails to solve in some scenarios, see:
https://www.youtube.com/watch?v=yHC8NQG2A2Y&t=4s @01:23
The text was updated successfully, but these errors were encountered: