X Tutup
Skip to content

Fix discrete key is not processed by advance_on_start in AnimationNode#114487

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
TokageItLab:fix-ad-start-animnode
Jan 6, 2026
Merged

Fix discrete key is not processed by advance_on_start in AnimationNode#114487
Repiteo merged 1 commit intogodotengine:masterfrom
TokageItLab:fix-ad-start-animnode

Conversation

@TokageItLab
Copy link
Member

Force process the key immediately after start by similar way with AnimationPlayer::_check_immediately_after_start().

@TokageItLab TokageItLab added this to the 4.6 milestone Dec 31, 2025
@TokageItLab TokageItLab requested a review from a team as a code owner December 31, 2025 21:50
@TokageItLab TokageItLab force-pushed the fix-ad-start-animnode branch 2 times, most recently from d08327c to 303ea06 Compare December 31, 2025 22:07
@TokageItLab TokageItLab force-pushed the fix-ad-start-animnode branch from 303ea06 to c8c0aa9 Compare December 31, 2025 22:26
Copy link
Contributor

@lyuma lyuma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it seems like a good change.

I feel a lot better knowing that the first keyframe (or last keyframe if PLAY_MODE_BACKWARD) is guaranteed to be processed in discrete animations rather than random behavior depending on the initial delta.

@Repiteo Repiteo merged commit 82bc54c into godotengine:master Jan 6, 2026
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Jan 6, 2026

Thanks!

rivie13 pushed a commit to rivie13/Phoenix-Agentic-Engine that referenced this pull request Feb 16, 2026
…nimnode

Fix discrete key is not processed by `advance_on_start` in AnimationNode
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Discrete key is not processed by advance_on_start in AnimationNode

3 participants

X Tutup