Collada export exporting -tive z scale as wrong x rotation #20634

New Issue

Stuart W · 2010-01-11T01:45:09+01:00

Stuart W commented

2010-01-11 01:45:09 +01:00

%%%Using build 25804..

In the attached .blend file is a cube modified to be assymetric, and the scaled -1 in Z at the model (not mesh) level.

When this is exported it exports the scale incorrectly (as a 3.1415 X rotation), and on collada re-import (or use in another tool) it is incorrect. If you look at the top view you will see the change between the blend file and the re-imported collada, we have verified by hand-checking the collada.

This created quite an issue as it breaks uv mapping assumptions for us, and was not like this in the previous python exporters.

We are using these exports in OSG, and the new collada exporter is a great improvement, keep up the great work.
%%%

%%%Using build 25804.. In the attached .blend file is a cube modified to be assymetric, and the scaled -1 in Z at the model (not mesh) level. When this is exported it exports the scale incorrectly (as a 3.1415 X rotation), and on collada re-import (or use in another tool) it is incorrect. If you look at the top view you will see the change between the blend file and the re-imported collada, we have verified by hand-checking the collada. This created quite an issue as it breaks uv mapping assumptions for us, and was not like this in the previous python exporters. We are using these exports in OSG, and the new collada exporter is a great improvement, keep up the great work. %%%

Stuart W commented

2010-01-11 01:45:09 +01:00

Changed status to: 'Open'

Stuart W commented

2010-01-11 02:03:17 +01:00

%%%Just digging a little deeper, it appears that mat4_to_size oversimplifies the matrix decomposition, therefore assuming orthonormal matricies, ie: rot, trans, +tive scale.
At a minimum a residual matrix needs to be calculated and allowed for.
%%%

%%%Just digging a little deeper, it appears that mat4_to_size oversimplifies the matrix decomposition, therefore assuming orthonormal matricies, ie: rot, trans, +tive scale. At a minimum a residual matrix needs to be calculated and allowed for. %%%

Stuart W commented

2010-01-22 06:41:15 +01:00

%%%
Is anyone going to look at this? this is quite a major bug in collada exports, without which a large number of belnder models are simply broken, and cannot be post-repaired.

%%%

%%% Is anyone going to look at this? this is quite a major bug in collada exports, without which a large number of belnder models are simply broken, and cannot be post-repaired. %%%

Arystanbek Dyussenov commented

2010-01-22 16:45:13 +01:00

%%%Hi Stuart,

I'm studying this problem now. Will try to fix today or tomorrow.

Thanks%%%

%%%Hi Stuart, I'm studying this problem now. Will try to fix today or tomorrow. Thanks%%%

Arystanbek Dyussenov commented

2010-01-23 13:53:03 +01:00

%%%Fixed in svn branch: https://svn.blender.org/svnroot/bf-blender/branches/soc-2009-chingachgook

Please check. I've detected a few other bugs that I want to fix before merging into trunk:

importer suffers from negative scale problem too
bones with negative scale are exported incorrectly too
object animation values are copied from FCurves which is not ok for child objects where values should be in parent-space

Thanks%%%

%%%Fixed in svn branch: https://svn.blender.org/svnroot/bf-blender/branches/soc-2009-chingachgook Please check. I've detected a few other bugs that I want to fix before merging into trunk: * importer suffers from negative scale problem too * bones with negative scale are exported incorrectly too * object animation values are copied from FCurves which is not ok for child objects where values should be in parent-space Thanks%%%

Stuart W commented

2010-01-23 23:10:43 +01:00

%%%
This is fantastic news Arystanbek, and much appreciated.

Unfortunately I wont be near a build environment for a couple of days but look forward to trying out these fixes.

Collada import/export is rapidly becoming essential for a lot of work so we are very happy to see these improvements.
%%%

%%% This is fantastic news Arystanbek, and much appreciated. Unfortunately I wont be near a build environment for a couple of days but look forward to trying out these fixes. Collada import/export is rapidly becoming essential for a lot of work so we are very happy to see these improvements. %%%

Stuart W commented

2010-02-02 22:47:01 +01:00

%%%So far looks good to me, do we need other testing before it gets integrated to main? we would love to see this mainlines ASAP.

Regards,
Stuart.%%%

%%%So far looks good to me, do we need other testing before it gets integrated to main? we would love to see this mainlines ASAP. Regards, Stuart.%%%

Arystanbek Dyussenov commented

2010-02-08 17:23:03 +01:00

%%%The exporter fix is already merged into trunk (revision 26444).

Still got to work on the related bugs I mentioned above. So please don't close this report.%%%

%%%The exporter fix is already merged into trunk (revision 26444). Still got to work on the related bugs I mentioned above. So please don't close this report.%%%

Nathan Letwory commented

2010-09-12 23:14:37 +02:00

%%%Arystan, any progress on these bugs?%%%

Nathan Letwory commented

2010-10-18 10:20:42 +02:00

%%%Currently not too much time to work on this, so I'm moving this to our todo list on the wiki: http://wiki.blender.org/index.php/Dev:2.5/Source/Development/Todo/Simple_Todos#COLLADA

Anyone who wants to help out still can provide patches!

Thanks for reporting.%%%

%%%Currently not too much time to work on this, so I'm moving this to our todo list on the wiki: http://wiki.blender.org/index.php/Dev:2.5/Source/Development/Todo/Simple_Todos#COLLADA Anyone who wants to help out still can provide patches! Thanks for reporting.%%%

Gaia Clary commented

2012-02-07 09:38:12 +01:00

%%%this report appears like it has been partially solved. Maybe it makes sense to create a new report for the remaining issues ? as from the log it is not precsisely clear (to me) what parts of the report are still broken.%%%

kanttori commented

2012-02-19 10:07:17 +01:00

%%%Hi,

This issue was most likely fixed with r43915 which changed export to use parentinv-matrix + loc/rot/scale instead of reinterpreting a matrix as loc/rot/scale. There are multiple "correct" ways to decomposite a matrix, but in this case if it's negative scale on single axis a rotation isn't sufficient to represent it so maybe a bug in decomposition. In any case, negative scales will remain problematic when decompositing is required in the importer.

%%%

%%%Hi, This issue was most likely fixed with r43915 which changed export to use parentinv-matrix + loc/rot/scale instead of reinterpreting a matrix as loc/rot/scale. There are multiple "correct" ways to decomposite a matrix, but in this case if it's negative scale on single axis a rotation isn't sufficient to represent it so maybe a bug in decomposition. In any case, negative scales will remain problematic when decompositing is required in the importer. %%%

Gaia Clary commented

2012-06-14 12:58:59 +02:00

%%%This issue seems to be fixed. Please ask for reopening if this is not true.%%%

Gaia Clary commented

2012-06-14 12:58:59 +02:00

Changed status from 'Open' to: 'Archived'

Gaia Clary closed this issue

2012-06-14 12:58:59 +02:00

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 export exporting -tive z scale as wrong x rotation #20634