Delgine 3D Tools & Content DeleD Community Edition
Forums
 
 FAQFAQ   SearchSearch   MemberlistMemberlist   UsergroupsUsergroups   RegisterRegister 
 ProfileProfile   Log in to check your private messagesLog in to check your private messages   Log inLog in 

Merge tool finally working 100%

 
This forum is locked: you cannot post, reply to, or edit topics.   This topic is locked: you cannot edit posts or make replies.    DeleD Community Edition Forum Index -> DeleD Development
View previous topic :: View next topic  
Author Message
chronozphere
DeleD PRO user


Joined: 20 Jun 2006
Posts: 1010
Location: Netherlands

PostPosted: Wed Feb 17, 2010 6:48 pm    Post subject: Merge tool finally working 100% Reply with quote

Chronozphere proudly presents:

...

A working merge tool! Very Happy

> Create the mesh at the left
> Select all its polygons
> Merge!

See the result:



Will be available in DeleD 2.43. Smile

@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. Smile
Back to top
View user's profile Send private message
Paul-Jan
Site Admin


Joined: 08 Aug 2004
Posts: 3066
Location: Lage Zwaluwe

PostPosted: Wed Feb 17, 2010 7:48 pm    Post subject: Reply with quote

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... Wink
Back to top
View user's profile Send private message Visit poster's website
chronozphere
DeleD PRO user


Joined: 20 Jun 2006
Posts: 1010
Location: Netherlands

PostPosted: Wed Feb 17, 2010 8:53 pm    Post subject: Reply with quote

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. Smile
Back to top
View user's profile Send private message
chronozphere
DeleD PRO user


Joined: 20 Jun 2006
Posts: 1010
Location: Netherlands

PostPosted: Wed Feb 17, 2010 10:05 pm    Post subject: Reply with quote

wow.. I am brilliant! Very Happy

I seem to have fixed the bug by moving some code around (I was playing lol Razz ) 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. Twisted Evil
Back to top
View user's profile Send private message
Dragonsbrethren
Member


Joined: 12 Apr 2009
Posts: 21

PostPosted: Wed Feb 17, 2010 11:51 pm    Post subject: Reply with quote

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
View user's profile Send private message Visit poster's website
Jeroen
Site Admin


Joined: 07 Aug 2004
Posts: 5332
Location: The Netherlands

PostPosted: Thu Feb 18, 2010 7:32 am    Post subject: Reply with quote

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
View user's profile Send private message Send e-mail Visit poster's website
Display posts from previous:   
This forum is locked: you cannot post, reply to, or edit topics.   This topic is locked: you cannot edit posts or make replies.    DeleD Community Edition Forum Index -> DeleD Development All times are GMT
Page 1 of 1

 
Jump to:  
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum