yvonneysf / opencollada

Automatically exported from code.google.com/p/opencollada
0 stars 0 forks source link

roundtrip ColladaMaya Maya plugin messes up scaling #21

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

Just load the attached file cylinder_skinned.mb

Use Maya 2009, with OpenCollada plugin version version 1.2.1 (32bit most 
recent from http://www.opencollada.org/download.html 2009-10-9)

Then File/Save All/Collada Maya .dae as cylinder_skinned.dae
Import this cylinder_skinned.dae again

Now the cylinder is 100 times too big, because
<scale sid="scale">100 100 100</scale>

Round trip should just work and not mess up units/scale etc.
Thanks,
Erwin

Original issue reported on code.google.com by erwin.coumans on 7 Nov 2009 at 2:15

GoogleCodeExporter commented 8 years ago
I suspect, it is due to the use of centimeter scale, which is ignored by the 
importer, so roundtrip fails (scales objects 100 times)

<unit name="centimeter" meter="0.01"/>

I would prefer to export COLLADA files using units in meters by default, and 
make 
this a setting in the COLLADA export settings, using meter="1.0" by default to 
avoid 
scaling issues.

Can you confirm?

Original comment by erwin.coumans on 9 Nov 2009 at 2:31

GoogleCodeExporter commented 8 years ago
We are about to investigate this issue and are not sure if it really a unit
conversion bug.

Yes, we can confirm.

Original comment by opencollada2@googlemail.com on 9 Nov 2009 at 3:21

GoogleCodeExporter commented 8 years ago
Has been fixed in rev 675.

Original comment by opencollada2@googlemail.com on 16 Nov 2009 at 9:10

GoogleCodeExporter commented 8 years ago

Really? I haven't tried it yet, but can you explain the fix?

I can only see a minor bugfix in rev 675 related to versioning:

>>else strcpy ( mayaVersion, DAE2MA::MAYA_VERSION_DEFAULT);

Thanks a lot,
Erwin

Original comment by erwin.coumans on 16 Nov 2009 at 1:10

GoogleCodeExporter commented 8 years ago
I have checked it again. It was already fixed in rev 664. 
But note that lot of changes came with this revision and the the relevant code 
had
been moved to the dae2ma project.

Original comment by opencollada2@googlemail.com on 16 Nov 2009 at 1:27