-
Notifications
You must be signed in to change notification settings - Fork 23
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
Selx 150 blueprint reader cannot handle two connections #152
Selx 150 blueprint reader cannot handle two connections #152
Conversation
added name to connection properties
"Component": { | ||
"Name": "ComponentB" | ||
}, | ||
"Connection": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! This style looks good
ConnectionPropertyType | ||
boost::multisetS, //OutEdgeList: allow multiple parallel connections | ||
boost::vecS, // VertexList, | ||
boost::bidirectionalS, //Directed: directed graph, but give access to traverse in both directions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens with the update order if we have a cyclic graph?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right, I forgot to handle that case in this PR. Currently, we don't have cyclic graphs, but I plan to work on the CompositeTransformComponent for multistage registration were that will happen. Is it ok with you if I address handling cyclic graphs there? (I have a cyclic graph check solution in mind that addresses other expected update issues with CompositeTransformComponent as well)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes that is fine :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blueprints are now able to have multiple parallel connections between 2 components.
To specify such connections the user must give the connections identifier names, which are by default "".
The networkbuilder was extended to handle the parallel connections.