godotengine / godot

Godot Engine – Multi-platform 2D and 3D game engine
https://godotengine.org
MIT License
89.18k stars 20.22k forks source link

Exported Curve fails to update with Project Running (live scene editing bug) #74697

Open aarongerber opened 1 year ago

aarongerber commented 1 year ago

Godot version

v4.0.stable.official [92bee43ad]

System information

Windows 10 64-bit operating Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz 4.01 GHz, Installed RAM 32.0 GB (31.9 GB usable)system, x64-based Vulkan API 1.3.205 - Forward Mobile - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 980 Ti

Issue description

Adjusting an exported Curve with the project running produces the following error:

E 0:00:22:0443 parse_message: Condition "p_args.size() < 10" is true. Returning: ERR_INVALID_DATA <C++ Source> scene/debugger/scene_debugger.cpp:174 @ parse_message()

https://github.com/godotengine/godot/blob/92bee43adba8d2401ef40e2480e53087bcb1eaf1/scene/debugger/scene_debugger.cpp#L174

The console output indicates the curve is not changing, and this is my experience with my game.

This type of feature seems to really need real-time behavior, and appeared to have it in 3.5 per this video: https://www.youtube.com/watch?v=gHT3jsCEiyA&t=241s

I am an extreme novice to GODOT programming. If I am misunderstanding the documentation here, then perhaps it needs to be updated: https://docs.godotengine.org/en/latest/classes/class_curve.html

Steps to reproduce

  1. Launch MRP (attached below)
  2. Run Project
  3. Adjust Curve

Note a Debug error occurs and the Console Output fails to change

Minimal reproduction project

CurvesFail.zip

Blue-Eclipse commented 1 year ago

Same issue here in Godot 4.0. Found the same video as the OP on using curves in Godot (3.5).

I'm using an exported curve for adjusting the brightness gradient in a shader. Trying to add points or moving them around in the curve while the project is playing gives the exact same error message as the OP found. Nothing gets modified in the project while its running.