sneakerpeet / opencollada

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

Patch / crash when exporting morphs in 3ds max #101

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Create an object with a morph controller. Important: The morph targets 
should not be nodes in the scenes which the morph controller references. 
Instead the morph controller should "own" the morph targets. 
2. Try to export the model in 3ds max 2010.

What is the expected output? What do you see instead?

Expected: Model is exported properly.
Actual: 3ds max crashes hard. This is because an assert is triggered at

ColladaMaxControllerExporter.cpp:123
assert( geometryExportNode );

Attached is a patch which solves the problem for me. I am not sure if this is 
the real fix or whether the real fix should be done somewhere else.

Patch discussion (2 small changes):

Change 1: The initial pose is not in the list of exported objects. Thus the 
assert in question fails. This should not happen. The patch adds code to check 
for this condition and inserts the initial pose into the list of exported 
objects if necessary.

Change 2: A small fix to properly escape morph target ids.

On demand I can provide an example file reproducing the problem.

Original issue reported on code.google.com by nitrogen...@gmail.com on 14 Aug 2010 at 2:57

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by opencollada2@googlemail.com on 25 Jan 2011 at 12:20