Fix #105088: FBX Camera Focus Distance is interpreted as millimeters #105124

Merged
Thomas Barlow merged 4 commits from Mysteryem/blender-addons:fix_105088_fbx_camera_focus_distance_mm into main 2024-01-16 21:48:22 +01:00
Member

FBX IO has been converting the Camera FocusDistance property from
millimeters on import and to millimeters on export, however, this
appears to have been incorrect because FocusDistance appears to be
in meters instead.

The FBX SDK used to have deprecated SetSpecificDistance() and
GetSpecificDistance() functions that were documented as accessing
Camera Focus Distance in millimeters.

The documentation of these functions appears to have been incorrectly
attributed to the Camera Focus Distance property itself, which could be
accessed directly without the deprecated functions, and appears to be
stored in meters.


The original discussion where the code was changed to use millimeters can be found at https://archive.blender.org/developer/D6666

FBX IO has been converting the Camera `FocusDistance` property from millimeters on import and to millimeters on export, however, this appears to have been incorrect because `FocusDistance` appears to be in meters instead. The FBX SDK used to have deprecated `SetSpecificDistance()` and `GetSpecificDistance()` functions that were documented as accessing Camera Focus Distance in millimeters. The documentation of these functions appears to have been incorrectly attributed to the Camera Focus Distance property itself, which could be accessed directly without the deprecated functions, and appears to be stored in meters. --- The original discussion where the code was changed to use millimeters can be found at https://archive.blender.org/developer/D6666
Thomas Barlow added 1 commit 2024-01-15 03:24:09 +01:00
FBX IO has been converting the Camera `FocusDistance` property from
millimeters on import and to millimeters on export, however, this
appears to have been incorrect because `FocusDistance` appears to be
in meters instead.

The FBX SDK used to have deprecated `SetSpecificDistance()` and
`GetSpecificDistance()` functions that were documented as accessing
Camera Focus Distance in millimeters.

The documentation of these functions appears to have been incorrectly
attributed to the Camera Focus Distance property itself, which could be
accessed directly without the deprecated functions, and appears to be
stored in meters.
Thomas Barlow requested review from Bastien Montagne 2024-01-15 05:44:01 +01:00
Bastien Montagne approved these changes 2024-01-15 12:27:22 +01:00
Thomas Barlow added 1 commit 2024-01-16 21:43:08 +01:00
Thomas Barlow added 2 commits 2024-01-16 21:47:11 +01:00
Thomas Barlow merged commit 2cdfe9e591 into main 2024-01-16 21:48:22 +01:00
Thomas Barlow deleted branch fix_105088_fbx_camera_focus_distance_mm 2024-01-16 21:48:24 +01:00
Sign in to join this conversation.
No reviewers
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#105124
No description provided.