Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable AnimationGraphPlayer to pass parameters #9

Merged
merged 7 commits into from
Dec 13, 2023

Conversation

mbrea-c
Copy link
Owner

@mbrea-c mbrea-c commented Dec 13, 2023

Objective

Currently the only way to set input parameters of a top-level graph (i.e. a graph assigned directly to an AnimationGraphPlayer) is to access the asset mutably and modify its parameter node. This would require cloning the asset every time we would like to have a new actor play the graph, which is inefficient. Ideally, we would set the parameters on AnimationGraphPlayer directly, and the player would then pass the parameters when querying the graph as a node overlay.

Solution

Enable setting parameters in the AnimationGraphPlayer, similarly to how GraphNode does it.

@mbrea-c mbrea-c added the enhancement New feature or request label Dec 13, 2023
@mbrea-c mbrea-c merged commit 92b6405 into master Dec 13, 2023
4 checks passed
@mbrea-c mbrea-c deleted the graph_player_parameters branch December 13, 2023 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant