graspologic icon indicating copy to clipboard operation
graspologic copied to clipboard

Published 20 hours ago • verified publisher icon graspologic-org

Inconsistent IO spec for node2vec

Open ebridge2 opened this issue 2 years ago • 1 comments

The entire package accepts adjacency matrices, but node2vec only accepts nx Graphs. This seems very undesirable since the package is not cohesive in this sense.

https://microsoft.github.io/graspologic/latest/_modules/graspologic/embed/n2v.html#node2vec_embed

ebridge2 avatar Mar 15 '22 15:03 ebridge2

Agreed, we had discussed moving node2vec out and into the pipeline package a while back but didn't get down to it.

daxpryce avatar May 23 '22 23:05 daxpryce

duplicate

bdpedigo avatar May 13 '24 19:05 bdpedigo