-
Notifications
You must be signed in to change notification settings - Fork 309
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
Updates nx-cugraph README.md with latest algos #4135
Updates nx-cugraph README.md with latest algos #4135
Conversation
…pdate README.md from nx_cugraph metadata directly.
I'm not sure I like the table view as is. It feels and looks like a raw data dump and is awkward to navigate and read. Ideally, if we want a table, I would want it to be interactive--sortable, filterable, etc. (but I don't know how to do this). Multiple "simple" tables would be better than a single table, such as three tables: 1. algorithms, 2. generators, 3. other/misc. I don't like mixing these if there's no way to unmix them. Did you try the tree view? What do you think of it? |
|
I don't know if we ever create and distribute Python source distributions (sdists), but I think we ought to allow sdists to be created correctly. To me, this means including more files such as This can be done by creating
This does not change the contents of wheels. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea to automate this.
…2-nx_cugraph_readme_update
…2-nx_cugraph_readme_update
@eriknw I pushed changes based on your suggestions here and resolved a merge conflict. We discussed bigger changes you wanted to make, which all seem good to me, but if you're not able to get them done in time we can now merge this as-is so the README is at least accurate for 24.02. |
…ratzel/cugraph into branch-24.02-nx_cugraph_readme_update
/merge |
closes #4079
README.md
with latest algosREADME.md
fromnx_cugraph
metadata directly