Blender Foundation: Blender 2.x BF release: Detail: 26475 <instance

Log In
New Account

[#26475] wrong in Collada export.

Date:
2011-03-12 16:56 Priority:
3

State:
Closed

Submitted by:
Juan Linietsky (reduz) Assigned to:
Nathan Letwory (jesterking)

Category:
None Status:
Closed

Relates to:
[#31362] Duplicates:

Patches:

Summary:
<instance_material> wrong in Collada export.

Detailed description

The symbol="" in <instance_material> is wrong, it must match the corresponding material="" in <triangles>,<polygons> or <poly_list>.
It is, instead, pointing to the material ID, so objects loaded have no material.

Followup

Message

Date: 2011-03-22 12:45
Sender: Imran Syed
Yo jester..

I made a couple of changes that seem to fix this issue. The bug reporter was 'goruka' on irc and I checked the collada spec and it is supposed to be as he says.

I am attaching the patch here. goruka reported that he does not build by himself so is unable to test export his own models/scenes for now. Maybe I'll get someone to make a build with the patch so he can test it (if necessary).

Date: 2011-03-22 12:47
Sender: Imran Syed
Added attachment?

Date: 2011-03-22 21:45
Sender: Nathan Letwory
Thanks! I'll be looking into the patch now.

Date: 2011-03-22 23:51
Sender: Nathan Letwory
Thanks for the report and patch. I've applied the patch and commited it in r35711.

Date: 2012-05-07 20:39
Sender: Gaia Clary
This fix breaks the Second Life Collada importer.

The reason is that different objects get different material symbols for the same material.
Although this seems to not violate the Collada specification, the Second LIfe importer
relies on the fact that identical materials use identical symbol names.

I have tested that the sequence number which was intriduced with commit 35711 is not needed.
I have added a patch for this a few days ago ( i just now found that this bug report causes the
problem). my patch is here:

http://projects.blender.org/tracker/index.php?func=detail&aid=31335&group_id=9&atid=127

I have already verified that materials keep intact after reimport of the exported dae files to blender.
Please can someone approve that my patch really creates collada conform output ?
Then my patch would fix the issues with Second Life as well.

thanks
Gaia

Date: 2012-05-08 01:00
Sender: Gaia Clary
I closed this again and opened a new bug report.
that is probably cleaner than to reopen an old report.

Attached Files:

Name	Date	Download

instance_material.patch	2011-03-22 12:48	Download

Changes:

Field	Old Value	Date	By

status_id	Open	2012-05-08 01:00	gaiaclary
close_date	None	2012-05-08 01:00	gaiaclary
Status	Reopened	2012-05-08 01:00	gaiaclary
close_date	2011-03-22 23:51	2012-05-07 20:39	gaiaclary
Status	Fixed / Closed	2012-05-07 20:39	gaiaclary
status_id	Closed	2012-05-07 20:39	gaiaclary
status_id	Open	2011-03-22 23:51	jesterking
close_date	None	2011-03-22 23:51	jesterking
Status	Investigate	2011-03-22 23:51	jesterking
File Added	15466: instance_material.patch	2011-03-22 12:48	freakabcd
assigned_to	none	2011-03-12 23:24	jesterking
details	The symbol="" in <instance_material> is wrong, it must match the corresponding material="" in <triangles>,<polygons> or <poly_list>. It is, instead, pointing to the material ID, so objects loaded have no material.	2011-03-12 23:24	jesterking
Status	New	2011-03-12 23:24	jesterking

Changes:

Blender 2.x BF release: Blender 2.6 Bug Tracker
[#31362] Collada: Naming convention for Material symbols breaks Second Life Importer (Relation: Relates to)