Support FBX Binary format

[takahirox]

This PR lets FBXLoader2 support FBX Binary format. #9860

Binary model data can include compressed data.
You need to import examples/libs/zlib_and_gzip.min.js for such models.

BTW does anyone have good FBX Binary model data which can be used as an example?

[mrdoob]

Many thankss!

[mrdoob]

Hmm, any chance this could depend on http://stuk.github.io/jszip/ instead? That's what https://github.com/mrdoob/three.js/blob/dev/examples/js/loaders/KMZLoader.js and https://threejs.org/editor/ depends on already.

[takahirox]

OK, I'll try.

(BTW who uses examples/libs/zlib_and_gzip.min.js?)

[mrdoob]

NRRDLoader.js it seems...

[takahirox]

Hm, it seems JSZip's APIs aren't fitting to FBXLoader.
FBXLoader just needs raw inflate(uncompress) function but
JSZip APIs expect zip (archived+compressed) file/data.

[mrdoob]

So then it mainly needs https://github.com/imaya/zlib.js/blob/master/bin/rawinflate.min.js, right?

What do you think about adding that code directly inside FBXLoader so it just works?

[takahirox]

I can try, but personally I think I want to separate them for maintenance.
(Maybe dev's ease of maintenance vs user's ease of use)

[mrdoob]

Oks! Sounds good 👌

[mrdoob]

Some clean up... NRRDLoader only needed gunzip, so I replaced zlib_and_gzip.min.js with gunzip.min.js. We already had inflate.min.js in the libs so we can use that for FBXLoader.

[vinaysshenoy]

The loader doesn't seem to handling certain FBX files properly. I've attached an FBX file which I'm using to test it, and the model imports correctly in Unity3D, but the geometry doesn't load properly in the FbxLoader. Attaching screenshots for reference.

Here's the FBX file

@takahirox Would you be able to take a look at this?

Note: The file unit is 1 unit = 1 mm, so you might want to adjust the camera far plane so you can see the entire object.

[takahirox]

Would you please check if text version of that model can be displayed correctly with FBXLoader?
I'd like to clarify if this issue is of binary parser.

[vinaysshenoy]

@takahirox I just checked. ASCII version is also not displaying correctly.

I suppose this is actually related to the Fbx -> THREE.Group conversion not handling certain cases.

[takahirox]

I see.
Would you please make another issue?
This thread already has been merged/closed.

[vinaysshenoy]

Sure, will do.