Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add basics for Blender support. #13

Merged
merged 1 commit into from Sep 29, 2013

Conversation

Projects
None yet
2 participants
Contributor

jesterKing commented Sep 24, 2013

This adds a Blender supporting script, a long with a necessary .blend file.

Several settings are added.

@jesterKing jesterKing Add basics for Blender support.
Note: blenderPath should be set by user to make sure it points to correct
installation.
0cf7e22
Contributor

fabrobinet commented Sep 24, 2013

Awesome. Thanks you. I'll try it over the WE before merging. Should I know anything in particular regarding install ? blender version... ?

Contributor

jesterKing commented Sep 25, 2013

Any recent build should work, I suggest you try one from http://builder.blender.org . Extract to the location of your choice, and don't forget to change the config.txt blenderPath setting for it.

We've been working with the Import, Export and Render steps only, as validating will fail currently anyway, since we have some blender specific profiles that aren't understood yet.

Many test cases will still fail, but we're using this to pinpoint the areas we need to work on and try to improve our support.

Contributor

fabrobinet commented Sep 25, 2013

Good to know, thanks - I'll try and get back to this PR.

Contributor

fabrobinet commented Sep 29, 2013

Hey @jesterKing I am running the CTS now. Many tests have been processed, I see the output dae and the image, that's awesome, great work !

A quick heads up: I just picked up randomly one test to try Test23 and step0/_reference_effect_out.dae.
There is one small issue preventing to validate the file, it is <library_controllers/> library controllers shouldn't be empty. Best is not exporting this it at all in that case. (There must be at least one controller - http://www.khronos.org/files/collada_spec_1_4.pdf page 98).

Right now, I might be stuck running a script, output is: start running script0.py so I'll wait a bit...

Contributor

fabrobinet commented Sep 29, 2013

Tests ended and I see results. Merging now.
Thanks again for your contribution !

FYI: While displaying the tests got some python issue complaining about memory but the symptoms on the console is messages about invalid image. Not sure if that's a CTS issue (due to to many objects in memory at the same time) or if that's a more a blender script issue. Anyhow since this doesn't prevent to get the whole CTS tests running and see the results, I am OK with merging as it is. That is tracked here : #14

col, type)
  File "Z:\code\COLLADA-CTS\Core\Gui\Grid\FImageRenderer.py", line 65, in __Draw
Image
    dx = image.GetWidth() - self.__imageWidth
  File "C:\Python24\Lib\site-packages\wx-2.6-msw-unicode\wx\_core.py", line 2578
, in GetWidth
    return _core_.Image_GetWidth(*args, **kwargs)
wx._core.PyAssertionError: C++ assertion "wxAssertFailure" failed in ..\..\src\c
ommon\image.cpp(820): invalid image
Traceback (most recent call last):
  File "Z:\code\COLLADA-CTS\Core\Gui\Grid\FImageRenderer.py", line 210, in Draw
    FImageRenderer.__CURRENT, type)
  File "Z:\code\COLLADA-CTS\Core\Gui\Grid\FImageRenderer.py", line 119, in __Tes
tAndDraw
    col, type)
  File "Z:\code\COLLADA-CTS\Core\Gui\Grid\FImageRenderer.py", line 65, in __Draw
Image
    dx = image.GetWidth() - self.__imageWidth
  File "C:\Python24\Lib\site-packages\wx-2.6-msw-unicode\wx\_core.py", line 2578
, in GetWidth
    return _core_.Image_GetWidth(*args, **kwargs)
wx._core.PyAssertionError: C++ assertion "wxAssertFailure" failed in ..\..\src\c
ommon\image.cpp(820): invalid image

@fabrobinet fabrobinet added a commit that referenced this pull request Sep 29, 2013

@fabrobinet fabrobinet Merge pull request #13 from jesterKing/blender
Add basics for Blender support.
39a3618

@fabrobinet fabrobinet merged commit 39a3618 into KhronosGroup:master Sep 29, 2013

Contributor

jesterKing commented Sep 29, 2013

Thanks for merging. Blender COLLADA support is indeed still lacking in places, but this should help our developers use the CTS for guiding their work on improving our support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment