search

pele-python / pele

Python energy landscape explorer
Other
95 stars 41 forks source link

sorting of nx connected components #136

Closed andyballard closed 9 years ago

andyballard commented 9 years ago

nx.connected_components(graph) does not automatically sort by size when running from a particular machine, and I think its the version of networkx being used: 1.9.1 vs 1.7. The below fix sorts explicitly.

js850 commented 9 years ago

thanks, networkx keeps changing how they do things which breaks our stuff.

The failing test seems like it's not related to this change. I restarted it, maybe it's one of the sporadically (and rarely) failing test. If it still fails some other dependency might have changed.

js850 commented 9 years ago

thanks Andy, I finally sorted the failing test and things work now. Merging

andyballard commented 9 years ago

great, thanks!