View previous topic :: View next topic |
Author |
Message |
chronozphere DeleD PRO user
Joined: 20 Jun 2006 Posts: 1010 Location: Netherlands
|
Posted: Wed Feb 17, 2010 6:48 pm Post subject: Merge tool finally working 100% |
|
|
Chronozphere proudly presents:
...
A working merge tool!
> Create the mesh at the left
> Select all its polygons
> Merge!
See the result:
Will be available in DeleD 2.43.
@Paul-jan: I'll upload the patch soon. First I need to tidy up some things.
EDIT: just discovered a small bug in it (merging complex geometry can have bad results).. It's too complex to fix, so I won't do that. A better approach might be to redesign the merge tool from scratch someday, but this will do for now. |
|
Back to top |
|
|
Paul-Jan Site Admin
Joined: 08 Aug 2004 Posts: 3066 Location: Lage Zwaluwe
|
Posted: Wed Feb 17, 2010 7:48 pm Post subject: |
|
|
Hey, awesome job, and kudos for adding the screenshot!
As long as the "complex problematic situations" are not too common and things didn't get much worse than they used to, I'll be more than happy to merge in your patch... |
|
Back to top |
|
|
chronozphere DeleD PRO user
Joined: 20 Jun 2006 Posts: 1010 Location: Netherlands
|
Posted: Wed Feb 17, 2010 8:53 pm Post subject: |
|
|
The complex problem i'm talking about is this:
> Create a cylinder
> Select a cap
> Inset that polygon twice
> Select all polygons that form the cap
> Merge.
Sometimes it works as it should, but sometimes it leaves two oddly-shaped polygons that cannot be merged. I believe this problem has always been there. |
|
Back to top |
|
|
chronozphere DeleD PRO user
Joined: 20 Jun 2006 Posts: 1010 Location: Netherlands
|
Posted: Wed Feb 17, 2010 10:05 pm Post subject: |
|
|
wow.. I am brilliant!
I seem to have fixed the bug by moving some code around (I was playing lol ) It's now possible to merge incredibly complicated surfaces.
After playing with the "new merge" I realized that there might be one thing missing. At this moment, merge creates concave polygons which is not desirable in most of the cases. Should we add another check to prevent this, or should this still be possible?
I also realized that I spend most of my DeleD development time on the merge tool. It's my territory now muhaha. |
|
Back to top |
|
|
Dragonsbrethren Member
Joined: 12 Apr 2009 Posts: 21
|
Posted: Wed Feb 17, 2010 11:51 pm Post subject: |
|
|
Hey, that's great. I'm working on something now that could definitely benefit from the improvement, since the merge tool currently isn't working too good for it. |
|
Back to top |
|
|
Jeroen Site Admin
Joined: 07 Aug 2004 Posts: 5332 Location: The Netherlands
|
Posted: Thu Feb 18, 2010 7:32 am Post subject: |
|
|
Good job! A correct working merge polygon tool is important to have. And indeed, it is complex material to get right, and I'm glad this is fixed now.
Concave polygons are something DeleD always offered and I think this should stay this way. After all, some users may actually want their geometry to be concave. If not, DeleD has the tools to make things convex anyway (Optimize). _________________ Check out Figuro, our online 3D app! More powerful 3D tools for free. |
|
Back to top |
|
|
|