BlenderTools icon indicating copy to clipboard operation
BlenderTools copied to clipboard

Send2UE: AnimationCurveNodeWrapper has no attribute 'add-keyframe'

Open DanMcLaughlin opened this issue 1 year ago • 9 comments

  • Addon: Send to Unreal
  • Steps to Reproduce the Problem: I clicked Pipeline > Export > Send to Unreal
  • Error Log: See below
  • OS: Windows
  • Addon Version: Send to Unreal 2.4.3
  • Blender Version: Blender 4.01
  • Unreal Version: Unreal 5.03

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 avatar Dec 02 '23 15:12 DanMcLaughlin

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

DanMcLaughlin avatar Dec 04 '23 13:12 DanMcLaughlin

@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 avatar Dec 12 '23 15:12 DanMcLaughlin

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 avatar Dec 24 '23 13:12 DanMcLaughlin

@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!

DanMcLaughlin avatar Jan 05 '24 13:01 DanMcLaughlin

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.

universalconquistador avatar Jan 11 '24 12:01 universalconquistador

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

DanMcLaughlin avatar Jan 11 '24 13:01 DanMcLaughlin

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 avatar Mar 05 '24 21:03 DooblyNoobly

btw fix worked for me @universalconquistador thanks!

DooblyNoobly avatar Mar 05 '24 21:03 DooblyNoobly

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?

JoshQuake avatar Mar 22 '24 01:03 JoshQuake