-
Notifications
You must be signed in to change notification settings - Fork 275
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
eachpoint(..., useLocalCoordinates=False)
manipulates the point's location when it shouldn't
#1099
Comments
greyltc
added a commit
to greyltc/cadquery
that referenced
this issue
Jun 8, 2022
…e object's location fixes CadQuery#1099
#1100 should bring the code in line with how I interpret the docs. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Here's what the docs say
eachpoint()
should do:cadquery/cadquery/cq.py
Lines 2440 to 2441 in a5fadeb
When
each()
is called withuseLocalCoordinates=False
, it does no manipulation of the object before passing it on to the callback function:cadquery/cadquery/cq.py
Line 2423 in a5fadeb
but when
eachpoint()
is called withuseLocalCoordinates=False
, it manipulates/transforms (p * loc
) the object's location before passing it on to the callback function:cadquery/cadquery/cq.py
Line 2479 in a5fadeb
I guess it probably shouldn't do that.
The text was updated successfully, but these errors were encountered: