AnimGraph.m_connection_data_storage size based on input sockets not number of connections. #6

New Issue

martin · 2023-04-01T14:07:32+02:00

martin commented

2023-04-01 14:07:32 +02:00

Currently the block AnimGraph.m_connection_data_storage is generally too big and scales with number of connections in the graph, instead of the number of output sockets.

However, a connection alwas ensures that the same address is used for the output and input sockets we waste space here.

Currently the block AnimGraph.m_connection_data_storage is generally too big and scales with number of connections in the graph, instead of the number of output sockets. However, a connection alwas ensures that the same address is used for the output and input sockets we waste space here.

martin added this to the Hierarchical Blend Tree project 2024-05-08 00:02:20 +02:00

martin added the

runtime

label 2024-05-08 00:14:23 +02:00

martin commented

2024-05-10 22:01:46 +02:00

Fixed as of cd56efca3d. Could still be improved / streamlined to make it easier to understand what actually ends up in AnimGraphBlendTree::m_input_buffer, AnimGraphBlendTree::m_output_buffer, AnimGraphBlendTree::m_connection_storage_buffer but will leave that for another cleanup in #14.

Fixed as of cd56efca3d53. Could still be improved / streamlined to make it easier to understand what actually ends up in `AnimGraphBlendTree::m_input_buffer, AnimGraphBlendTree::m_output_buffer, AnimGraphBlendTree::m_connection_storage_buffer` but will leave that for another cleanup in #14.

martin closed this issue

2024-05-10 22:01:46 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: martin/AnimTestbed#6