Merge Co-planar Faces for STL Import about sketchup-stl HOT 20 CLOSED

Personally, I think we should "just do it" for all imports. Other thoughts?

from sketchup-stl.

Scott, the DWG and 3DS imports works poorly like that. I never use those options because they often end up losing faces. These is a mismatch between what the importers think is coplanar and what SketchUp thinks.

My plugin CleanUp works quite well and I can port it into this project, but there are a couple of extreme edge cases where it can fail. For that reason I suggest to keep it as an option.

If your willing to work out these edge cases it'd be best though. I could port that back to Cleanup and maybe you could make some nudges to make the DWG and 3DS imported work properly as well..

from sketchup-stl.

The only reason I can think of not to cleanup after importing is possibly to debug the importer or test the validity of the .stl file.

My plugin CleanUp works quite well and I can port it into this project

Best answer!

from sketchup-stl.

And I'm sure there are some users that will find some use of keeping the original triangulation. You know there always will be.

from sketchup-stl.

This repo need some sample files for testing. Can someone with some nice variations of STL files push a folder with some test models?

from sketchup-stl.

Is there a way to add test files to a github repository like this or shall
I just share a DropBox folder?

On Fri, Oct 26, 2012 at 3:32 PM, Thomas Thomassen
[email protected]:

This repo need some sample files for testing. Can someone with some nice
variations of STL files push a folder with some test models?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/12#issuecomment-9827763.

Jody

from sketchup-stl.

Should be possible to upload binary files. So I'd think a folder with STL files should work. But maybe hold on until other people chime in - maybe they don't want binary files in the repo... ??
To it'd make sense so we'd all have easy access to the same test files.

from sketchup-stl.

I'll be adding some stuff as I find it here:
https://www.dropbox.com/sh/gm8sv1vzcrrp0yb/lzLsnLV-82

On Fri, Oct 26, 2012 at 3:40 PM, Thomas Thomassen
[email protected]:

Should be possible to upload binary files. So I'd think a folder with STL
files should work. But maybe hold on until other people chime in - maybe
they don't want binary files in the repo... ??
To it'd make sense so we'd all have easy access to the same test files.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/12#issuecomment-9827947.

Jody

from sketchup-stl.

Is merging co-planar faces the only type cleanup that will be needed on imported stl files? I don't think we know yet.

from sketchup-stl.

Detect tiny holes?

from sketchup-stl.

My point was where is the line drawn as far as including features? My own response is we need to test and refine the importer before we can determine how much cleanup will be needed.

from sketchup-stl.

For the importer, there's probably not much cleanup past coplanar stuff. I'll agree that fixing colpanar faces should be an option since we already have a UI button for just such things. Might as well give the user the control. It doesn't clutter the UI much.

The export is where cleanup will get interesting... Lots of possibilities there.

from sketchup-stl.

I see there is a system to assign an issue to a user - I was looking to set myself as assignee for this issue to indicate I'm working on it. Just thought it be useful so we avoid working simultaniously on the same fix. Anyone knows how it works?

from sketchup-stl.

I think only admins can assign issues to people. Not sure if collaborators can. Workarounds I have seen are to claim it in the comments then later on an admin can assign it. I added you to my fork as a collaborator as a test to see if you can assign issues on that fork.

from sketchup-stl.

I'm not finding any UI in the browser. When I use the Android app there is a button, but the list of people to assign is empty.

from sketchup-stl.

The UI on the Web when you are an admin is a button next to the top of an issue where it say "No one is assigned" In the case of this repo I see nothing, in the case of my fork I see a button where I can assign it to collaborators or anyone by name.

from sketchup-stl.

Yea, I see the button in your fork.

from sketchup-stl.

Cool, so if we can just get Scott to add us all as collaborators we will be good to go :)

from sketchup-stl.

I cannot close this issue?

from sketchup-stl.

Hrm. I guess I'll close it then.

from sketchup-stl.