-
Notifications
You must be signed in to change notification settings - Fork 103
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
179 - more graph operations #230
base: master
Are you sure you want to change the base?
Conversation
✅ Doxygen Documentation successfully built!
To edit notification comments on pull requests, go to your Netlify site configuration. |
c493447
to
3895122
Compare
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.
Looks good to me, can merge once CI passes and you give your ok
void join(Graph& G1, const Graph& G2, NodeArray<node>& nodeMap) { | ||
OGDF_ASSERT(nodeMap.valid()); | ||
void join(Graph& G1, const Graph& G2, NodeArray<node>& mapping) { | ||
OGDF_ASSERT(mapping.valid()); |
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.
Instead of (or maybe in addition to) valid()
I'd rather check mapping.graphOf() == &G2
it("creates an edge where there was none", [&]() { | ||
complement(G, false, false); | ||
edge edge12 = G.searchEdge(n1, n2); | ||
AssertThat(edge12, Is().Not().Null()); |
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.
There's also !IsNull()
if that is more convenient to you...
bef8075
to
ea5557b
Compare
Adding additional graph operations. These include
complement
,intersection
andjoin
.