-
Notifications
You must be signed in to change notification settings - Fork 168
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
bool - holter tower #53
Comments
Hey @vegarringdal can you confirm that this still happens? Cheers! |
Nope, but noticed npm version and console is printing out different version... |
Hey @vegarringdal yeah that's actually a mistake on our side, you are probably using 35, the log has a typo. If you could provide me the model privately, we'll address this asap. Cheers! |
I just downloaded a random holter tower IFC file from internet. 😁 |
If this is 169 MB file from https://www.markusplassen.no/?p=159 - these windows are now fine. Although there is still a 100+ boolean problems in the file: edit: I think they are fine because the walls around them are gone 😅 |
Hey @makc so still problematic, huh? We'll need to take a deeper look into this when we release more bounties regarding booleans! |
I'd love to know more about what's going on here. Are you referring to merging nearby verts to e.g. make an STL manifold? Like most things, the simple cases are simple, but making it robust can be pretty tricky. Certainly pass any Speaking of, I had no idea you took me on as a dependency; that's exciting! Ideally I'd like to keep a list somehow of what projects are using the library, though it seems easier said than done. Anyway, thanks for mentioning me. |
@elalish I could log input mesh if you are willing to take a look at that in the context of polygon.cpp error;
currently ifc input data is just a triangle soup - so I have attempted to merge the vertices back before passing to manifold lib and when we have a simple enough mesh it works. but I guess there are cases where too many vertices touch and then merging them is doing nothing but harm. |
Yeah, this is why triangle soup like STL is such a problem - it's basically impossible to reliably retain manifoldness in general cases. This is why I helped develop the 3MF format, which is focused on defining and retaining manifoldness. Yes, please log the input meshes you're having trouble with. If you can manage to write a PR with a |
Hey @elalish sorry for not letting you know, my bad. We will probably release more bounties to identify these errors, fix the ones related to geometry generation on our side and support your fantastic work (like we did with Garret's three-mesh-bvh), if that's ok with you. 🙂 We'll let you know of any news. In the meantime, of course, feel free to include IFC.js as one of the projects using your library as a dependency, and let us know if you need anything from us. |
Fantastic, keep those bug reports coming! And certainly let me know if have any comments on the API. |
From what I can see this is now rendering correctly. So I will close this |
Holter tower is missing a few windows
Tried this with v 0.20 with and without USE_FAST_BOOLS
The text was updated successfully, but these errors were encountered: