Previously we were limiting the nodes included during the omnibus embedding to be the set of nodes in the intersection of the largest connected components between each pair of graphs passed as an argument.
With some joint research with JHU we have developed a new strategy that allows us to embed more nodes. To accomplish this, we:
Create a graph that contains the union of all edges across all time series graphs
Calculate the LCC of that union graph
For each time series graph, assert that:
a. Remove all nodes not contained in the vertex set of the LCC union graph
b. Add all nodes in the LCC union graph to the graph as an isolate (no edges)
This is important as it allows us to maintain the correct shape of each time series graph's adjacency matrix
I added a test called test_union_lcc_returns_expected_shape that hopefully illuminates the exact scenario.
Previously we were limiting the nodes included during the omnibus embedding to be the set of nodes in the intersection of the largest connected components between each pair of graphs passed as an argument.
With some joint research with JHU we have developed a new strategy that allows us to embed more nodes. To accomplish this, we:
I added a test called
test_union_lcc_returns_expected_shape
that hopefully illuminates the exact scenario.