Export error with lights blender and babylon js

[Johgold]

Hello, 

 

I'm using Blender 2.7.2 with babylon js, I'm trying to export a musicbox (coming from a .obj) in .babylon but when I drop it in the babylon sandbox the lights don't work, they're not the same as in blender.

 

If anybody know a solution that be great, you can a capture of what I get in the sand box below. 

 

Thank you 

[GameMonetize]

What kind of light are you using? Can you share your blender scene?

[Johgold]

I'm using an "hemi" light, I can't share my blender scene because I tried to make a .zip with the scene and texture but the file is to heavy to be upload :/

[JCPalmer]

DK, that is a bug that I discovered.  Hemi lights are pointing in the wrong direction due to an incorrect sign of last dimension, z I thing. I have had no luck figuring it out.  The original exporter  had:

        else:            matrix_world = object.matrix_world.copy()            matrix_world.translation = mathutils.Vector((0, 0, 0))            direction = mathutils.Vector((0, 0, -1)) * matrix_world            Export_babylon.write_vector(file_handler, "direction", -direction)

The multi-pass is:

        else:            matrix_world = light.matrix_world.copy()            matrix_world.translation = mathutils.Vector((0, 0, 0))            self.direction = -(mathutils.Vector((0, 0, -1)) * matrix_world)

Do you know why the sign ends up wrong?  I have tried a number of ways, but no joy.

 

Johgold,  you can manually change the sign in your .babylon file as a temp fix.

[GameMonetize]

Ok let me fixed the exporter

[GameMonetize]

Can you try with the latest version I updated??

[JCPalmer]

That did it.  That stray '-' in Export_babylon.write_vector(file_handler, "direction", -direction)  must not have been doing anything, and when I re-factored it, implemented in a way where it did.

 

I just tested in TOB.  There are now un-needed ().  I was in the process of diffing the 2 today, in Netbeans.  I have changes for you.  Either code you are not using, but did not remove, comments, or improved log entries.  Will put in pull request & take care of parens + update version #.

 

Jeff

[GameMonetize]

thanks!!

[JCPalmer]

Have made changes locally, and just going to turn this into a reconciliation thread.  Set version to 1.3, because I think it was at 1.2 before.  Currently pointing to 1.1, which was TOB's at time of copy.

 

You did take out the option to have an animation start at other than frame 0.  Gryff specifically asked for that.  Cannot remember the exact thread, but it is out there somewhere. 

 

Here are a series of diff snaps.  io_export_babylon.py is on the left, & io_tower_of_babel.py on the right. 

 

The FCurveAnimatable constructor comments out & forces auto animate to 0  - end (You did not delete get_first_frame() & get_last_frame).  Yes, I added it being conditional on my side.

 

Later in VectorAnimation constructor, you build up the frames the old way.

 

Finally, when TOB did both .babylon & .js / .ts, I was storing skeleton indices 2 different ways.  I added comments to that effect.  Wondering if each side should only do the version they write out, since armatures are kind of slow?

 

 

 

 

 

 

 

 

[GameMonetize]

So:

The FCurveAnimatable constructor comments out & forces auto animate to 0  - end: You can revert to your version here. Wrong copy/paste

Later in VectorAnimation constructor, you build up the frames the old way. I think you can revert this one as well

Finally, when TOB did both .babylon & .js / .ts,: COuld you jsut keep the .babylon way for io_export_babylon.py?

 

Thank you for your appreciated help!

[JCPalmer]

Will do sometime today.

[JCPalmer]

Well, all changes are done for both, except that I thought I might test it.  Blend files are in extensions repository https://github.com/BabylonJS/Extensions/tree/master/Morph/Blender/blender-test/blender-in

 

First, I used camera_anim.blend to test that making autoanimate switchable in TOB, and that the animation reversion of official exporter back to TOB both worked.  They do.

 

I then used armature.blend to test each only builds the skeleton indices in the format which they actually write out.  That works.  Did notice that now the direction for the hemi light in this .blend has the wrong sign for the middle axis.  I did the hemi direction fix testing with voice_sync.blend, which is for MORPH 1.1, and not in the repository yet.  The light was very close to the ground & pointed horizontally, so this did not show up.

 

I cannot easily fix this myself, because I do not really know what that code does.  DK, Please look at issue with armature.blend.  Sandbox should work to see the light problem.  The skeleton is queued to walk in html, so it will just stand there.

 

When we get a final verdict on how to make the sign correct, make that change only and submit.

 

Jeff

[GameMonetize]

Here is what I get so far. Is it the expected result?

[JCPalmer]

Yes.  I do not know why, but in my tests I now get the stomach in light & chest dark, like light coming from below now. 

"lights":[{"name":"Hemi","id":"Hemi","type":3,"direction":[0,-1,0],"groundColor":[0,0,0],"intensity":1,"diffuse":[1,1,1],"specular":[0,0,0]}],

switched to 0,1,0, then I get what you get.  I am stumped.  Maybe those parens are needed.  I have to go. Will pick up tomorrow.

[GameMonetize]

Thanks!

[JCPalmer]

Just to make sure I have an accurate working setup to isolate the delta from working to not, you are Blender 2.72b, the io_export_blender.py currently in repository, and the .blend from the extensions respository, right?

[JCPalmer]

I commit what I had & put in a pull request, version 1.3.0.  I could not reproduce your success with Blender 2.7.2b, the same .blend, and the direction line unmodified.  It could be my other changes, but I doubt it.

 

If I cannot establish a close delta of success to failure, I cannot fix this without knowing what this matrix_world, the setting of it's translation, and multipling by a 0,0,-1 vector is supposed to be doing.  Sorry.  Recommend making a version 1.3.1 that fixes it.

[Johgold]

Thank you for your advices, it almost look like I wanted it. The Lights are working now. 

 

Thanks 

 

Johgold