Skip to content

Add node to computation graph by ID

dawehr requested to merge comp_graph_add_id into master

Added ability to create nodes with arbitrary IDs. This should only be used if necessary, preferring graph_add_node over graph_add_node_id, because it does not use hashing, and must allocate an array up to the size of the ID you requested.

The ability to add nodes by ID is necessary for the ground station to use the existing graph infrastructure for showing nodes. We could have left the requirement such that after all nodes are added, they must be consecutive, because that is what the quadcopter will return. I am not a fan of this, though, because it seems like something that will bite teams in the future if they don't know about that unexpected behavior.

Merge request reports