thomasp85 / tidygraph

A tidy API for graph manipulation
https://tidygraph.data-imaginist.com
Other
546 stars 61 forks source link

rm igraph::estimate_ functions that will soon be deprecated #174

Closed maelle closed 11 months ago

maelle commented 1 year ago

Cf https://github.com/igraph/rigraph/issues/844

szhorvat commented 1 year ago

Just noticed something small which @thomasp85 might want to fix at the same time:

centrality_closeness() has cutoff=NULL as default while centrality_betweenness() has cutoff=-1. The meaning is identical, but for consistency, it would be nice to use NULL for both.

@thomasp85, I also wanted to let you know that igraph has an implementation of harmonic centrality with cutoffs. You might consider using this for centrality_closeness_harmonic().

thomasp85 commented 11 months ago

Thanks, Maelle!