EpicGamesExt / BlenderTools

Blender addons that improve the game development workflow between Blender and Unreal.
https://epicgamesext.github.io/BlenderTools/
MIT License
2.76k stars 49 forks source link

Send2UE: AnimationCurveNodeWrapper has no attribute 'add-keyframe' #696

Open DanMcLaughlin opened 10 months ago

DanMcLaughlin commented 10 months ago

Something is different in 4.01 - a lot of breaking changes with the Python API, I'm seeing it in other addons. But also this python error isn't showing up in the error log, I'm only finding it in the popup which I grabbed an image of

The relevant error is in core/io/fbs.py line 159 AnimationCurveNodeWrapper has no attribute 'add-keyframe'. Did you mean 'set_keyframes'?

image

DanMcLaughlin commented 10 months ago

BTW - delete any animation data and it transfers. This issue is for an asset with a skeleton obviously.

DanMcLaughlin commented 9 months ago

@james-baber the plugin appears to be pretty broken with the latest Blender, we're unable to use it, can you check on this? Thanks -

DanMcLaughlin commented 9 months ago

So if you actually need to transfer something with animation you're stuck obviously ... I tried changing those calls to 'set_animations' but then get another error

image

File "C:\Program Files\WindowsApps\BlenderFoundation.Blender_4.0.2.0_x64__ppwjx1n5r4v9t\Blender\4.0\scripts\addons\io_scene_fbx\fbx_utils.py", line 1294, in set_keyframes
    if len(keyframe_values.shape) == 1:
AttributeError: 'Vector' object has no attribute 'shape'

So I'm out of my depth as to what this code is doing and the issue

DanMcLaughlin commented 8 months ago

@james-baber or @iigindesign - any traction on this? With all the animation changes in Blender 4.0 the addon does look broken and I need some help on this one - we're stuck without this working, thanks!

universalconquistador commented 8 months ago

I reconstructed the fbx_animation_do function based on the latest 4.0 FBX plugin, and it seems to work so far, but someone should double check my work before merging it in #706.

DanMcLaughlin commented 8 months ago

@universalconquistador I just tested your code; works fine for me! Good catch that this was originally just borrowing the fbx exporter code.

DooblyNoobly commented 6 months ago

Why is this still a problem 2 months later. Can we get a review and merge of the fix asap. @james-baber or @iigindesign

DooblyNoobly commented 6 months ago

btw fix worked for me @universalconquistador thanks!

JoshQuake commented 6 months ago

Why is this still a problem 2 months later. Can we get a review and merge of the fix asap. @james-baber or @iigindesign

The lack of support here is quite disheartening. :/

@ryan-hummer Are you able to assist in merging the fix?