pomegranate: incompatibilities with the upcoming networkx 2.0

networkx 2.0 switches graph.nodes() to return a generator rather than a list (see https://networkx.readthedocs.io/en/latest/reference/release_2.0.html#api-changes), so constructs such as len(self.graph.nodes()) (https://github.com/jmschrei/pomegranate/blob/master/pomegranate/hmm.pyx#L693) will fail with

TypeError: object of type 'generator' has no len()

About this issue

  • Original URL
  • State: closed
  • Created 8 years ago
  • Comments: 20 (6 by maintainers)

Most upvoted comments

Pretty sure this should have been closed by #551, I’ve been using nx 2 w/o issue

What breaks when NetworkxX 2.0 is installed? Many of my pipelines are dependent on nx >= 2.0 at the moment 😕

@jmschrei any idea whether you’ll pull @poplarShift’s fix in? seems like a reasonable workaround to me 🤷‍♂️

any progress on this ?