thomasp85 / tidygraph

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

Do not use 'nobigint' parameter for betweenness() #155

Closed szhorvat closed 1 year ago

szhorvat commented 2 years ago

Starting with igraph 1.3, the nobigint parameter of betweenness() is ignored and triggers a warning. It will be completely removed from the next version. It makes sense to also remove it from tidygraph.

In the past, this option provided a tradeoff between speed and precision. Betweenness calculations are now more precise than either with or without nobigint in past versions, and just as fast as with nobigint=T.

jdfoote commented 1 year ago

I created a very simple patch in #163

szhorvat commented 1 year ago

This was fixed by #163.