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

Make WASM path configurable + Update IFC library #21683

Merged
merged 3 commits into from
Apr 20, 2021
Merged

Make WASM path configurable + Update IFC library #21683

merged 3 commits into from
Apr 20, 2021

Conversation

agviegas
Copy link
Contributor

@agviegas agviegas commented Apr 19, 2021

Related issue: #21673

Description

This solves the route to the WASM file as a setter of IfcLoader, so the end user can put it wherever they prefer. Also, I have updated the web-ifc library to the latest version.

@agviegas
Copy link
Contributor Author

You can find a working example here (see app.js).

@Mugen87
Copy link
Collaborator

Mugen87 commented Apr 19, 2021

It seems the example is not working anymore:

https://raw.githack.com/agviegas/three.js/IfcLoader/examples/webgl_loader_ifc.html

Uncaught SyntaxError: Unexpected token ')'

@mrdoob mrdoob added this to the r128 milestone Apr 19, 2021
@agviegas
Copy link
Contributor Author

It is working on my end now. Let me know if there is anything else to correct!

@mrdoob mrdoob merged commit 8722959 into mrdoob:dev Apr 20, 2021
@mrdoob
Copy link
Owner

mrdoob commented Apr 20, 2021

Thanks!

@mrdoob
Copy link
Owner

mrdoob commented Apr 20, 2021

The trees are no longer red (and e2e didn't caught this...) 🤔

Screenshot 2021-04-20 at 08 11 39

@agviegas
Copy link
Contributor Author

Hum, probably the trees are not red anymore because of the last advances regarding IFC material reading (I think that red was a default case). If it's not urgent, I can update the shot of the example in the next PR (which will be soon).

@mrdoob
Copy link
Owner

mrdoob commented Apr 20, 2021

No worries! I'll update the screenshot now.

@mrdoob
Copy link
Owner

mrdoob commented Apr 20, 2021

be73dd5

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

Successfully merging this pull request may close these issues.

3 participants