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

Geometry differences between web-ifc demo page and OBC.IfcLoader #504

Open
5 tasks done
robertbiskup opened this issue Oct 1, 2024 · 2 comments
Open
5 tasks done
Labels
bug Something isn't working

Comments

@robertbiskup
Copy link

Describe the bug 📝

Hi @agviegas and @HoyosJuan, could you please take a look at the issue disscused at ThatOpen/engine_web-ifc#1057 ? We got different results after uploading ifc files to web-ifc demo page(https://thatopen.github.io/engine_web-ifc/demo) and engine-components viewer using OBC.IfcLoader ( https://unpkg.com/[email protected]/58 and "@thatopen/components-front": "2.2.2",
"@thatopen/components": "2.2.11"). Web-ifc demo page all ifc files shows correctly - when using components - geometry errors appear. There are sample screenshots and ifc files.

Reproduction ▶️

No response

Steps to reproduce 🔢

Load ifc in newer components

System Info 💻

@thatopen/components-front": "2.2.2",
@thatopen/components": "2.2.11"

Used Package Manager 📦

npm

Error Trace/Logs 📃

No response

Validations ✅

  • Read the docs.
  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • Make sure this is a repository issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
  • Check that this is a concrete bug. For Q&A join our Community.
  • The provided reproduction is a minimal reproducible example of the bug.
@robertbiskup robertbiskup added the bug Something isn't working label Oct 1, 2024
@HoyosJuan
Copy link
Member

Hey @robertbiskup! That's beacuse we haven't update the web ifc dependency in components yet. Will be donee for release 2.3.0 coming this week.

@robertbiskup
Copy link
Author

@HoyosJuan Thank you for the quick answer! This is crucial information, I thought that setting the "wasm" parameter was enough, so we are waiting for the update! ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants