[Plugin] CleanUp

thomthom

Armed with more info from Simone Nicolo I now have a better method for detecting coplanar faces - the same method Sketchup use. I think it's 100% reliable. No more messing about with tolerances.

Highly recommended update!

thomthom

@thomthom said:

Armed with more info from Simone Nicolo I now have a better method for detecting coplanar faces - the same method Sketchup use. I think it's 100% reliable. No more messing about with tolerances.

Highly recommended update!

Turns out it's not 100%. In one of my test model it still left some edges that could be erased.

Updated with some minor UI fixes.

EscapeArtist

"The selected attachment does not exist anymore"

Error when clicking for download.

thomthom

@escapeartist said:

"The selected attachment does not exist anymore"

Error when clicking for download.

You probably tried to download just as I uploaded a quick patch.

CadFather

Thanks Thom, this is good.

EscapeArtist

@thomthom said:

You probably tried to download just as I uploaded a quick patch.

Ah, timing is everything. Works just fine. Thanks!

MtnTrails

Thank you, Thom! Works just like I'd hoped. What a great time saver!

Gaieus

Hi Thom,

Thanks for the update. I'll definitely try it out (I managed to forget about two coplanar faces with differently positioned but the same texture again and screwed up my model - no big hassle though just a "doh" moment again...)

thomthom

@gaieus said:

Hi Thom,

Thanks for the update. I'll definitely try it out (I managed to forget about two coplanar faces with differently positioned but the same texture again and screwed up my model - no big hassle though just a "doh" moment again...)

Ah yes - I'm still not sure how to account for that. But it is possible ... somehow...

Rich O Brien

Hi Thom,

Thanks for the update, one of my favourite plugins.
Doesnt say much about my modelling ability!

thomthom

Important update in 1.2.2
Sorry for all these updates recently.
I think I found the root of why the last update wasn't 100%.

It now erases edges as they are processes, instead of in bulk later. This is because each face's plane equations must be re-evaluated after it's been merged. This might lead to slightly longer processing time - but I now hope that it truly is 100% watertight.

EarthMover

Thanks for the update. I use this puppy all the time!

Jim

Thanks - it is doing better job of removing edges. I am willing to trade a little extra time for a cleaner model.

I did get a reference to a deleted Edge error on this model (.stl import.) I haven't investigated the cause, but I've attached it in case you're curious.

Error: #<TypeError: reference to deleted Edge> C:\Program Files\Google\Google SketchUp 7\Plugins\tt_cleanup.rb:283:infaces'`

My fix:
283: next unless e.valid? and e.is_a?(Sketchup::Edge)

But is doesn't explain the cause.

moai.skp

thomthom

hm.. that is odd. I wonder why that happens. thanks for posting the model.

thomthom

Updated to account for the issue reported by Jim.

Though I still don't understand why I ever get a reference to a deleted edge...

TIG

Trap unexpected deleted edges with edge.erase! if edge.valid? type of code ?

thomthom

Yes - but what puzzles me is why the code comes across deleted references at all. After reviewing my code I can't work out why. I am iterating an array copy of the entities collection. And I only iterate the array once.

thomthom

Gaieus: This update is for you. Made it investigate the UV's before merging.
Not that it is very strict, the UV values for either vertex of the edge has to be identical for each face connected to it before it merges. This means there might be cases where it visually looks like they should merge, but doesn't. They should merge is you position a material on one face and use the dropper to paint other adjacent faces.
To turn of this feature just enable the Ignore UV's.

Note that this is pretty fresh code that hasn't been tested well. (Only very quickly.) So please investigate the model after using the tool and report any unexpected behaviour. (Please most sample model as that will make debugging much easier.)

Ern

Very handy,

Now all the cookies are yours!

Gaieus

@thomthom said:

Gaieus: This update is for you. Made it investigate the UV's before merging...

Thanks a bunch, Thom,
I will definitely check it out. Lately I have needed this function really badly.