ufbx: bake_fps option in importer / FBXDocument api has no effect.

[lyuma]

Tested versions

Only in 4.3.dev

System information

Godot v4.3.dev (2efbc6b) - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3090 (NVIDIA; 31.0.15.3713) - Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz (16 Threads)

Issue description

The ufbx code does not use the bake_fps option. This is because the API takes p_bake_fps in the generate_scene function, but not append_from_file where it is needed.

If FBXDocument were a standalone class, we could change the API, but because it inherits from GLTFDocument, we can't change this API.

My recommendation is to add an extra bake_fps property in FBXState and assign this in the editor_scene_importer_ufbx. I'm not sure what to do about the bake_fps argument to generate_scene since it has no effect.

Steps to reproduce

1. Import an fbx with a 60 fps animation.

2. Select FBX2glTF and set fps to 60.

3. Look at the Animation. it will show extra keyframes:
   

4. Select ufbx and set fps to 60.

5. The animation will only have 30 frames per second. The same happens when using a FPS below 30 as well.
   

Minimal reproduction project (MRP)

I do not have any redistributable 60fps FBX file.