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 

Collada exporter bug (fixed!)

 
Post new topic   Reply to topic    DeleD Community Edition Forum Index -> DeleD Development
View previous topic :: View next topic  
Author Message
CableGuy
Member


Joined: 11 Jan 2011
Posts: 14

PostPosted: Sun May 13, 2012 9:45 pm    Post subject: Collada exporter bug (fixed!) Reply with quote

Hey guys,

I think I've found a bug in the Collada exporter. Good news are I've looked at the source and know how to fix it (just a simple fix really). Bad news are I haven't got any Delphi compiler to test it.

Does anyone here has access to a Delphi compiler?
Back to top
View user's profile Send private message
CableGuy
Member


Joined: 11 Jan 2011
Posts: 14

PostPosted: Wed May 16, 2012 3:42 pm    Post subject: Reply with quote

Does this mean the project has officially zero active developers?
Back to top
View user's profile Send private message
Jeroen
Site Admin


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

PostPosted: Wed May 16, 2012 3:47 pm    Post subject: Reply with quote

There are a number of people who have a Delphi compiler, but appearantly nobody is motivated to dive into the Collada exporter (which is understandable, given the daunting nature of the Collada format imho).

I am, however, willing to give it a try if you know exactly where I need to fix things? Let me know!
_________________
Check out the gallery, our DeleD Wiki and Youtube videos!
Back to top
View user's profile Send private message Send e-mail Visit poster's website
CableGuy
Member


Joined: 11 Jan 2011
Posts: 14

PostPosted: Wed May 16, 2012 8:20 pm    Post subject: Reply with quote

Sure.

The file is in /trunk/collada/Source/Applications/Exporter/Exporter/DCColladaWriterCl.pas

lines 493 & 494 read:
aWriter.AppendProperty( 'input_semantic', 'TEX_COORD' );
aWriter.AppendProperty( 'input_set', 0 );

but should be:
aWriter.AppendProperty( 'input_semantic', 'TEXCOORD' );
aWriter.AppendProperty( 'input_set', 1 );

The change is that the input set should be 1 and according to the collada spec it is TEXCOORD and not TEX_COORD

Thanks for your time.

PS: I totally agree that the Collada format is daunting and over engineered but imho is the only format that can be safely used as an interchange format (3ds and obj can get you so far...)
Back to top
View user's profile Send private message
Jeroen
Site Admin


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

PostPosted: Fri May 18, 2012 7:31 am    Post subject: Reply with quote

Ok, that should be easily fixable. Smile Will let you know when done!
_________________
Check out the gallery, our DeleD Wiki and Youtube videos!
Back to top
View user's profile Send private message Send e-mail Visit poster's website
Jeroen
Site Admin


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

PostPosted: Fri May 18, 2012 7:48 am    Post subject: Reply with quote

Ok, fixed. Can I send you the dll for testing purposes?
_________________
Check out the gallery, our DeleD Wiki and Youtube videos!
Back to top
View user's profile Send private message Send e-mail Visit poster's website
CableGuy
Member


Joined: 11 Jan 2011
Posts: 14

PostPosted: Fri May 18, 2012 8:41 am    Post subject: Reply with quote

Sure. You can send it to ibachar _at_ gmail _dot_ com
Thanks.
Back to top
View user's profile Send private message
Jeroen
Site Admin


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

PostPosted: Fri May 18, 2012 9:20 am    Post subject: Reply with quote

Ok, I've sent you a RAR file containing the fixed collada exporter. Let me know if this works for you!
_________________
Check out the gallery, our DeleD Wiki and Youtube videos!
Back to top
View user's profile Send private message Send e-mail Visit poster's website
Jeroen
Site Admin


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

PostPosted: Sun May 20, 2012 2:19 pm    Post subject: Reply with quote

Confirmed fixed. Smile
_________________
Check out the gallery, our DeleD Wiki and Youtube videos!
Back to top
View user's profile Send private message Send e-mail Visit poster's website
AWM Mars
Member


Joined: 06 Jan 2010
Posts: 1193
Location: Wilts England

PostPosted: Sun May 20, 2012 5:58 pm    Post subject: Reply with quote

Excellent work Smile

Is this fixed version now linked to the Plug-in page?
_________________
Politeness is priceless when received, cost nothing to own or give, yet some cannot afford.

Checkout:
http://www.awm.mars.yourinside.com/
http://www.bccservices.co.uk
http://www.localtradecheck.co.uk
Back to top
View user's profile Send private message Visit poster's website
Display posts from previous:   
Post new topic   Reply to topic    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