Collada: fix broken export of transforms (and adds new bake transforms option) #34062

New Issue

Daniel Balster · 2013-01-30T22:37:14+01:00

Daniel Balster commented

2013-01-30 22:37:14 +01:00

%%%This patch fixes the COLLADA export of blender (svn rev ~54214) where each node has redundant/broken transforms:

    <node ...>
      <matrix sid="transform">...</matrix>
      <rotate sid="rotationZ">...</rotate>
      <rotate sid="rotationY">...</rotate>
      <rotate sid="rotationX">...</rotate>
      <scale sid="scale">...</scale>
      <translate sid="location">...</translate>
    </node>

If "transform" is already present, rot/scale/translate are not necessary. Additionally, it should be translate -> rotate -> scale to get the correct results.

This patch adds a new option "bake transforms" (default off) that lets you export all transforms as a single matrix, if set. If unset, you get the old behavior (translate, rotate, scale).

Verification:

just export the default blender scene (cube,camera,light) as COLLADA with default options. Open the resulting untitled.dae file and search for

    <node id="Cube" name="Cube" type="NODE">
      <translate sid="location">0 0 0</translate>
      <rotate sid="rotationZ">0 0 1 0</rotate>
      <rotate sid="rotationY">0 1 0 0</rotate>
      <rotate sid="rotationX">1 0 0 0</rotate>
      <scale sid="scale">1 1 1</scale>

now export with "bake transforms" checked and you'll get

    <node id="Cube" name="Cube" type="NODE">
      <matrix sid="transform">1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 1</matrix>

%%%

%%%This patch fixes the COLLADA export of blender (svn rev ~54214) where each node has redundant/broken transforms: ``` <node ...> <matrix sid="transform">...</matrix> <rotate sid="rotationZ">...</rotate> <rotate sid="rotationY">...</rotate> <rotate sid="rotationX">...</rotate> <scale sid="scale">...</scale> <translate sid="location">...</translate> </node> ``` If "transform" is already present, rot/scale/translate are not necessary. Additionally, it should be translate -> rotate -> scale to get the correct results. This patch adds a new option "bake transforms" (default off) that lets you export all transforms as a single matrix, if set. If unset, you get the old behavior (translate, rotate, scale). Verification: - just export the default blender scene (cube,camera,light) as COLLADA with default options. Open the resulting untitled.dae file and search for ``` <node id="Cube" name="Cube" type="NODE"> <translate sid="location">0 0 0</translate> <rotate sid="rotationZ">0 0 1 0</rotate> <rotate sid="rotationY">0 1 0 0</rotate> <rotate sid="rotationX">1 0 0 0</rotate> <scale sid="scale">1 1 1</scale> ``` - now export with "bake transforms" checked and you'll get ``` <node id="Cube" name="Cube" type="NODE"> <matrix sid="transform">1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 1</matrix> ``` %%%

Daniel Balster commented

2013-01-30 22:37:14 +01:00

Changed status to: 'Open'

Gaia Clary commented

2013-02-11 23:47:07 +01:00

%%%I have fixed the duplicate transforms by removing the trans/rot/scale version, see revision 54477.

I am not sure if we should add an option for "bake transforms"
Can you give an example where that would be needed ?%%%

%%%I have fixed the duplicate transforms by removing the trans/rot/scale version, see revision 54477. I am not sure if we should add an option for "bake transforms" Can you give an example where that would be needed ?%%%

Daniel Balster commented

2013-02-12 08:13:47 +01:00

%%%I'd prefer explicitly named transforms like 0 0 1 0 instead of a baked matrix. COLLADA is a verbose exchange format, allowing the user to spot and find each bit she's interested in. This is why I proposed an option for this - I'm very happy with rot/loc/scale, but if others want matrices they have an option now.

Animations can also target these named transforms, I'm not sure if baking would break this system.%%%

%%%I'd prefer explicitly named transforms like <rotate sid="rotationZ">0 0 1 0</rotate> instead of a baked matrix. COLLADA is a verbose exchange format, allowing the user to spot and find each bit she's interested in. This is why I proposed an option for this - I'm very happy with rot/loc/scale, but if others want matrices they have an option now. Animations can also target these named transforms, I'm not sure if baking would break this system.%%%

Brendon Murphy commented

2015-09-08 04:49:58 +02:00

Closed as duplicate of #43298

Brendon Murphy closed this issue

2015-09-08 04:49:58 +02:00

Campbell Barton commented

2015-09-08 05:32:37 +02:00

Changed status from 'Duplicate' to: 'Open'

Campbell Barton reopened this issue

2015-09-08 05:32:37 +02:00

Gaia Clary commented

2019-05-27 18:44:26 +02:00

Changed status from 'Open' to: 'Resolved'

Gaia Clary closed this issue

2019-05-27 18:44:26 +02:00

Gaia Clary commented

2019-05-27 18:44:26 +02:00

@dbalster The Blender 2.80 exporter now has 2 options for exporting transforms either as "Matrix" or as "Decomposed". The restrictions from older Blender releases no longer apply.

I believe the feature makes this patch obsolete. Please tell me if you do not agree. Then we can reopen this task.

cheers,
Gaia

@dbalster The Blender 2.80 exporter now has 2 options for exporting transforms either as "Matrix" or as "Decomposed". The restrictions from older Blender releases no longer apply. I believe the feature makes this patch obsolete. Please tell me if you do not agree. Then we can reopen this task. cheers, Gaia

Mick Pearson commented

2019-05-29 02:39:52 +02:00

Added subscriber: @Mick-P-2

Mick Pearson commented

2019-05-29 02:39:52 +02:00

For the record, if matrix and the others are present, then both should be applied by the COLLADA software. It's not a multi-representational option in other words. Second, regarding:

Additionally, it should be translate -> rotate -> scale to get the correct results.

I don't know how Blender's transforms work, but traditionally translation is the last step. When you multiply a vector with a matrix the final column (or row) does translation, and it's simply to add its value to the vector.

For the record, if matrix and the others are present, then both should be applied by the COLLADA software. It's not a multi-representational option in other words. Second, regarding: > Additionally, it should be translate -> rotate -> scale to get the correct results. I don't know how Blender's transforms work, but traditionally translation is the last step. When you multiply a vector with a matrix the final column (or row) does translation, and it's simply to add its value to the vector.

blender-admin commented

2019-05-29 08:57:47 +02:00

This issue was referenced by 820e4d4303

This issue was referenced by 820e4d4303906020136b9a81ee0e878caee1f199

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Collada: fix broken export of transforms (and adds new bake transforms option) #34062