search

ThatOpen / engine_web-ifc

Reading and writing IFC files with Javascript, at native speeds.
https://thatopen.github.io/engine_web-ifc/demo
Mozilla Public License 2.0
617 stars 190 forks source link

[Bug]: Issue with geometry #426

Open rkaeplive opened 1 year ago

rkaeplive commented 1 year ago

What happened?

I have issues with geometry T-geometry is wrong rotated (1st picture- OpenIFCViewer, 2nd - ifcjs) image image

and some problems with back side image image

Version

"web-ifc-viewer": "^1.0.217"

What browsers are you seeing the problem on?

No response

Relevant log output

No response

Anything else?

No response

rkaeplive commented 1 year ago

geometry_issues.Ifc.zip

QuimMoya commented 1 year ago

I opened your model in IFC.JS image

And then in Bimvision image

While there may be an issue with the expansion of metallic profiles, I have been unable to locate the specific problem you mentioned earlier.

QuimMoya commented 1 year ago

Hi, good news, it seems last update solved some issues.

image

rkaeplive commented 1 year ago

@QuimMoya what version of ifcjs do you use? I use npm install. I can send you the repo of this issue

rkaeplive commented 1 year ago

@QuimMoya https://github.com/rkaeplive/geometry-issue-three in this case this T-geometry is rotated image and some bugs with meshes image

QuimMoya commented 1 year ago

@rkaeplive Ok i will look at this

QuimMoya commented 1 year ago

The problem with rotated T profiles will be solved in the next release 0.0.42 , this week we will do it. image But the problem of missing faces is still present in some profiles, i will work on it.

rkaeplive commented 1 year ago

Hello, did you solve this?

rkaeplive commented 1 year ago

@QuimMoya hello?

QuimMoya commented 1 year ago

@rkaeplive

Hello

Sorry for the delay. Yes, the problem of rotated profiles is solved. Not sure if all problems with missing faces are solved.

rkaeplive commented 1 year ago

@QuimMoya did you release it ?

rkaeplive commented 1 year ago

Hello again, did you solve missing faces issue?

QuimMoya commented 1 year ago

@rkaeplive

Yes, the missing faces should be solved, and the rotated T. We will do a new release soon and many of these problems should became solved But if you find anything wrong please post the picture, it's easier for us to identify problems in the model using pictures

rkaeplive commented 1 year ago

@QuimMoya hello again Where are many missing faces here, im not use DoubleSide material

image

backside is ok

image

this file Clinic_A.ifc.zip

QuimMoya commented 2 weeks ago

Is it solved now?

image

Please open it with the demo

Confirm it open Ok and i will close the issue