-
Notifications
You must be signed in to change notification settings - Fork 298
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)
has a coordinate transformation bug
#1098
Comments
Try |
The above is a contrived example specifically to illustrate the bug, not to achieve any final geometry that I want. @lorenzncode, are you aware of any usage of "global XY coordinates" are (0,0,0) at the center of the axis cross. (A) doesn't put the boxes there, so why is it correct? |
Is it incorrect or not according to your expectations @greyltc ? |
@adam-urbanczyk @lorenzncode This issue is just meant to give an example of what is fixed by #1097 I believe @greyltc is saying that the current behavior does not match expectations. |
I've tried to keep my expectations of how I think I'll post a separate issue on how I believe there's a mismatch between what the documentation says and what the code actually does when |
I've made #1099 which covers my expectations based on the docs and how |
I'm not sure that the current implemntation is correct, but what you write contradicts with the screenshot @greyltc . |
@adam-urbanczyk You mean what I wrote in #1099 contradicts what's here? #1099 is meant to be an entirely separate issue from this one (they just happen to occur on the same line). Solving #1099 (in the way I imagine it should be solved, as in #1100) makes this whole issue (and its solution) obsolete. If you agree that #1099 describes a real problem, then probably ignore this issue. If you disagree that #1099 is a problem, then consider this issue. |
The
Maybe I got it wrong and @greyltc is correct to expect the boxes to vary in Z here. |
On the contradiction: @greyltc you suggest in one of the comments that the objects should be centered around (0,0,0), but it is clearly not the case in the screenshot B. Could you give a definitive description of what is the desired behavior? AFAICT screenshot A shows the correct behavior and B is just |
For example, the output of
is (A)
but it should be (B)
Luckily #1097 fixes it!
The text was updated successfully, but these errors were encountered: