search

networkx / networkx-metis

NetworkX Addon to allow graph partitioning with METIS
Other
79 stars 21 forks source link

Now runs with new networkx #61

Closed Lamaun closed 4 years ago

Lamaun commented 4 years ago

The change of networkx graph data structure broke this package.

dschult commented 4 years ago

This repository hasn't been actively updated in a while. Probably the python versions tested on Travis should match the current networkx versions. It is possible that other deprecated functionality from NetworkX could cause problems too. Thanks for doing this!

Lamaun commented 4 years ago

I haven't worked with travis yet, should we replace travis.yml with something similar to https://github.com/networkx/networkx/blob/master/.travis.yml?

dschult commented 4 years ago

Can you try a more simple change first? I'm hoping it doesn't require the more heavy lifting.

Change it to python versions 3.6, 3.7, 3.8, pypy3 and let's see what Travis says. Thanks!

Lamaun commented 4 years ago

Seems like that works

dschult commented 4 years ago

Excellent -- anything else before we merge?

Lamaun commented 4 years ago

Not from my side. Could we get this into pypi?

dschult commented 4 years ago

I'll look into getting it into pypi.