Inconsistency in Node API #225
Comments
|
Good catch. Will fix. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
{{ message }}
If we look at the Node API, we have:
def randomOutboundNodeSet(numResults: Int, rnd: Random): Seq[Int]:https://github.com/twitter/cassovary/blob/master/cassovary-core/src/main/scala/com/twitter/cassovary/graph/Node.scala#L123
def outboundNodes(): CSeq[Int]:https://github.com/twitter/cassovary/blob/master/cassovary-core/src/main/scala/com/twitter/cassovary/graph/Node.scala#L89
Note that one returns a
CSeqand the other returns aSeq. Should this be made consistent?The text was updated successfully, but these errors were encountered: