OGRECave / blender2ogre

Blender exporter for the OGRE 3D engine
GNU Lesser General Public License v2.1
215 stars 70 forks source link

OgreMeshTool Fails #78

Open MBetters opened 3 years ago

MBetters commented 3 years ago

Addon settings:

image

Export settings:

image

Error:

image

MBetters commented 3 years ago

The mesh tool is valid. It's from ogre-next (v2):

image

MBetters commented 3 years ago

My config.py:

image

MBetters commented 3 years ago

My exit code for OgreMeshTool isn't 0.

image

MBetters commented 3 years ago

Disabling both Shape Animation and Shape Normals is what worked. Duplicate of https://github.com/OGRECave/blender2ogre/issues/56

sercero commented 3 years ago

@MBetters they are actually different issues, the add-on scripts are failing at different instances.

In issue #56 the error is in mesh.py, now the error seems to be in util.py when calling the OgreMeshTool.

Since I'm not using Ogre 2.x, I didn't test this scenario when recommending that you disable only Shape Normals.

sercero commented 3 years ago

Can you please reopen this issue?

Thanks.

sercero commented 3 years ago

@MBetters it seems that the problem is trying to use OgreMeshTool with pose animations and the v2 option, I think that pose animations are not supported in Ogre v2 Items.

I will ask @darksylinc in the forums.

paroj commented 3 years ago

maybe for now we should just gracefully abort with "Pose animations are currently not supported for V2 meshes". This would hint to the workaround instead of a cryptic error message.

sercero commented 3 years ago

@paroj I was going to ask what do you think about not using an assert if the tool returns non-zero value, but just printing an error in the log and the report saying that the binary mesh will not be generated, just the xml.

paroj commented 3 years ago

when I added the asserts, exporting XML only was not an option. Should be fine nowadays.

sercero commented 3 years ago

@paroj another question, I was thinking of adding an option to delete the .xml after conversion. Do you concur?