Skip to content
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

Error in some entities location #47

Open
babybeibeili opened this issue Jul 7, 2020 · 7 comments
Open

Error in some entities location #47

babybeibeili opened this issue Jul 7, 2020 · 7 comments

Comments

@babybeibeili
Copy link

Some location error and i can't fix it.
Can you help me fix it or give me some advice? thanks!
normal
image
error
image

left5.zip

@Serthys
Copy link
Contributor

Serthys commented Jul 7, 2020

This is usually a block problem. If you can edit the files you can try grouping all object in the existing block or removing all object from the block.

@bzuillsmith I thought blocks were supported, is this some edge case?

@bzuillsmith
Copy link
Member

There is some support for blocks. In this case it appears there is a rotation happening and I believe there was an issue I never got around to fixing with rotations.

@babybeibeili
Copy link
Author

image
I change Upward Direction <0.0,0.0,-1.0> to <0.0,0.0,1.0> ,display as follows
image
But I don't know how to modify this error.

@bzuillsmith
Copy link
Member

Hmmm I'm not great at understanding three.js translations, rotations, and scaling. The code for that is here. It's possible the order in which these operations are performed matters and that I did something out of order. Maybe I should be adding the entities, then rotating, then translating? Another possibilitity is that the block has a base point which I don't think I added support for. This would cause translations to be wrong.

@babybeibeili
Copy link
Author

@bzuillsmith
Copy link
Member

Oh, I see. I did not totally understand your post about changing the extrusion direction at first. I understand now. You may be right. Does it work correctly if you change it to 0,0,1 and the Z-Scale to 1?

@babybeibeili
Copy link
Author

If extrusion direction is (0,0,-1) ,I will change the roation and position, it work correctly.But I don't know if this is suitable for other files.

The extrusion direction specified by the group codes 210, 220 and 230 is by default (0,0,1). This is the normal case. Extrusion direction (0,0,-1) is also supported. In this case, the shapes will be mirrored at the Y axis.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants