Skip to content

Standardize use of AbstractGraph.n_nodes and add n_edges

Vicentini Filippo requested to merge github/fork/femtobit/graph-api into master

Created by: femtobit

Addressing one of the issues raised by @PhilipVinc in #514 (closed) with my suggestion of standardizing on n_nodes (also adding n_edges in the process).

Slight benefits compared to n_vertices:

  1. n_nodes and n_edges have the same length.
  2. networkx also uses the term nodes.

Slight downsides:

  1. n_nodes is used a lot in the code to refer to MPI processes. This is unlikely to cause much confusion, though.

Merge request reports