Skip to content

Conversation

@greghogan
Copy link
Contributor

Graph algorithms for annotating

  • vertex degree for undirected graphs
  • vertex out-, in-, and out- and in-degree for directed graphs
  • edge source, target, and source and target degree for undirected graphs

greghogan added 6 commits May 2, 2016 12:15
Graph algorithms for annotating
  vertex degree for undirected graphs
  vertex out-, in-, and out- and in-degree for directed graphs
  edge source, target, and source and target degree for undirected graphs
This is an optimization for VertexDegreePair which need only perform a
single join and for the edge annotations which do not need zero degree
vertices.
for algorithms which, for example, use an edge weight.
@greghogan greghogan force-pushed the 3772_graph_algorithms_for_vertex_and_edge_degree branch from 66f6f0e to 3126efd Compare May 2, 2016 18:05
@greghogan greghogan force-pushed the 3772_graph_algorithms_for_vertex_and_edge_degree branch from 57ff7e4 to 45c4724 Compare May 3, 2016 16:59
<td><strong>TranslateGraphIds</strong></td>
<td>degree.annotate.directed.<br/><strong>VertexInDegree</strong></td>
<td>
<p>Annotate vertices of a directed graph with the in-degree count.</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean by in-degree "count"? What's the result of this operation?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am removing "count" from the descriptions since "degree" implies a count.

@vasia
Copy link
Contributor

vasia commented May 6, 2016

@greghogan, after your last commit, there are several gelly test cases failing on travis. Could you please take a look? Thanks!

@greghogan
Copy link
Contributor Author

@vasia I had looked at one of the failed tests and it wasn't due to bad logic but rather to running out of some minimal number of memory buffers. This results, I expect, from updating the Graph API's degree functions to translate the output of the degree algorithms. I think it is best to leave the Graph API as-is for now.

@vasia
Copy link
Contributor

vasia commented May 9, 2016

I've only left a minor comment @greghogan, otherwise I think it's good to merge.

@greghogan
Copy link
Contributor Author

Merging ...

@greghogan
Copy link
Contributor Author

Closed in 70181b2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants