[Plugin] CleanUp

[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,

---

[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.

[j40]

Thanks a lot Thom !! This is one of the greatest plugins.

[thomthom]

Version 2.0.0

• Supports ThomThom's Menu
• Persistent preferences.
• Settings for global and local cleanup synced.
• Repairs split edges. (Based on Carlo Roosen's Repair Broken Edges)
• Removes duplicate faces.
• Fixes duplicate component definition names.
• Runs Validity Check at the end.

"Duplicate faces" are faces where multiple faces share the same set of vertices. The plugins should also be able to detect smaller faces overlapping larger faces.

[Gaieus]

Downloading...

[Rich O Brien]

Hi Thom,

thanks for the update

Incredible tool! Should be mentioned in the Visual Index thread

Rich

[Jim]

Yep, this is the best cleanup script available.

I know the choice of user interface is limited, but I'm not a fan of nested menus. I was wondering if a webdialog interface would work. So then I could have a single shortcut that would bring up the "cleanup" dialog showing the options available, and then "run" from there. Maybe? Someday?

Edit: "best" is an under-statement - it's important and essential.

[thomthom]

@jim said:

I know the choice of user interface is limited, but I'm not a fan of nested menus. I was wondering if a webdialog interface would work. So then I could have a single shortcut that would bring up the "cleanup" dialog showing the options available, and then "run" from there. Maybe? Someday?

Yes - that is exactly what I have in mind. The options are growing so a better UI is required. However, I'm trying to prioritise Vertex Edit. This has been a necessary distraction because I need it at work.

So my plans for v2 that did not make it will be postponed to v3:

• Merge Identical and similar materials
• Axis adjustments
• Find identical groups/components (If I can find a good algorithm for this.)
  For this webdialog UI is a must in order to be friendly to use.

Thanks for letting me know how to activate the Fix Model function - in some cases it appears that erasing big bulk of edges messes up SUæs Loops. Even though everything looks fine in SU. So it was good to force that test after a cleanup.

[solo]

This is one of my most cherished ruby's, thank you for improving it and making it an invaluable tool.

[Gaieus]

@thomthom said:

...So my plans for v2 that did not make it will be postponed to v3:

• Merge Identical and similar materials
• Axis adjustments
• Find identical groups/components (If I can find a good algorithm for this.)
  ...

Now these definitely sound incredible! Can hardly wait to see them in here!

[gilad75]

Thanks a lot, you created magic!

[Harbor]

THanks a lot man, It should be one of the best plugin of SU

[davery]

Thanks. Greatly appreciated.