fbx2json.py crashes when parsing fbx files with a version number = 7500 (I haven't tried others) #49822

Closed
opened 2016-10-23 15:04:11 +02:00 by Raul Aguaviva · 8 comments

I actually fixed the script to support these new version fbx's, you can diff with yours to see what the fix is

fbx2json.py

Cheers!

I actually fixed the script to support these new version fbx's, you can diff with yours to see what the fix is [fbx2json.py](https://archive.blender.org/developer/F384951/fbx2json.py) Cheers!
Bastien Montagne was assigned by Raul Aguaviva 2016-10-23 15:04:11 +02:00
Author

Changed status to: 'Open'

Changed status to: 'Open'
Author

Added subscriber: @xchip

Added subscriber: @xchip

Thanks for the fix, but this is not acceptable as-is, since you are therefore breaking all pre-7500 files with those changes (sizes should be switched after having read version imho).

Anyway, this is trivial to fix, but more importantly here, did you try that change in actual parsing code, and then importing a .fbx file in Blender?

Thanks for the fix, but this is not acceptable as-is, since you are therefore breaking all pre-7500 files with those changes (sizes should be switched after having read version imho). Anyway, this is trivial to fix, but more importantly here, did you try that change in actual parsing code, and then importing a .fbx file in Blender?
Author

I said I fixed but I meant that I found out what the issue with the new version is. What I did was a quick change to show you where the problem is.

I said I fixed but I meant that I found out what the issue with the new version is. What I did was a quick change to show you where the problem is.

OK, will take over from there and try and see if this is enough to make new format work.

OK, will take over from there and try and see if this is enough to make new format work.

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'

Thanks looks like that was the only issue with new FBX indeed, so now Blender can import FBX2016 too. :)

Thanks looks like that was the only issue with new FBX indeed, so now Blender can import FBX2016 too. :)
Author

My pleasure! Thanks to you for your prompt support :)

My pleasure! Thanks to you for your prompt support :)
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#49822
No description provided.