Skip to content

Add network/graph category to scientific domains tab #374

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Dec 31, 2020

Conversation

jarrodmillman
Copy link
Member

Fixes gh-320. See #313 (comment).

@rgommers
Copy link
Member

Thanks Jarrod. Having another look around for other packages that may deserve a mention, what do you think about these:

They all seem solid/popular/maintained, and it's not clear to me how they compare against graph-tool and PyGSP.

@mdeff
Copy link
Contributor

mdeff commented Dec 29, 2020

NetworkX, graph-tool, igraph, NetworKit, snappy, scikit-network do network science: they are concerned with the analysis of networks. PyGSP, which stands for Graph Signal Processing, is more concerned with the analysis of data / features / signals supported by networks (as functions of vertices) and is closer to (spectral) graph theory.

I have the following to say about the NetSci category:

  • NetworkX is definitely the most popular
  • graph-tool is well polished and maintained by its author, more mature and probably a bit more popular than NetworKit
  • scikit-network is too new (first commit a year ago)
  • snap and their Python SWIG interface (snappy) seems mostly used internally (by Leskovec's group at Stanford)
  • igraph and NetworKit are both solid complements to NetworkX and graph-tool
  • igraph is however more popular (e.g., 257,455 vs 29,153 downloads from conda-forge, or dependent projects on github and libraries.io) and mature (first commit 14 vs 7 years ago).

I see two alternative categories depending on whether or not you want to take the alternative PyGSP point-of-view:

  1. Network Science: NetworkX, graph-tool, igraph, NetworKit
  2. Graphs and Networks: NetworkX, graph-tool, igraph, PyGSP

I'd vote for 2, but as the author of PyGSP I'm of course biased.

@rgommers
Copy link
Member

Thanks for the thorough overview @mdeff

I'd vote for 2, but as the author of PyGSP I'm of course biased.

I'm happy to go with this option. "Graphs and Networks" does sound better to me.

@jarrodmillman
Copy link
Member Author

I updated the PR to go with option 2.

I also slightly changed the icon. I committed the script that generates it:

  • static/images/content_images/sc_dom_img/graph.py

Feel free to change the colors, line thickess, or whatever. Or let me know if you want me to change something.

Is there a better place to put the script? Should I take it out of the repo?

@jarrodmillman
Copy link
Member Author

I just looked #369 and noticed it looks like they got rid of the colors and the grays. I'll change the icon again.

@rgommers
Copy link
Member

Adding the script next to the icon sounds fine to me. Would it be possible to make the png size smaller though @jarrodmillman? The new icon is 10x larger than the other icons, 250 kb is a lot for a small image.

@mdeff
Copy link
Contributor

mdeff commented Dec 30, 2020

Why not export as SVG? Should be small given the small number of elements. I think they're moving to SVG in gh-369.

@jarrodmillman
Copy link
Member Author

I saved the icon as a SVG instead of a PNG. It is now 8.0K.

I also made it 160px by 120px. I couldn't find any specification of the preferred size, so I just randomly selected one of the existing PNGs and used its width.

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM now, thanks @jarrodmillman!

@rgommers rgommers merged commit b296142 into numpy:master Dec 31, 2020
@rgommers rgommers changed the title Add network/graph category Add network/graph category to scientific domains tab Dec 31, 2020
@jarrodmillman jarrodmillman deleted the network branch December 31, 2020 18:48
MarsBarLee pushed a commit to MarsBarLee/numpy.org that referenced this pull request Feb 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ecosystem: graphs and networks?
3 participants