- Notifications
You must be signed in to change notification settings - Fork5
Add logo and benchmarks to README#60
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Changes fromall commits
f392d0d1271e71b8ee0a3dda122ddb9cbc30850421e840706File filter
Filter by extension
Conversations
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,9 @@ | ||
| recursive-include graphblas_algorithms *.py | ||
| prune docs | ||
| include setup.py | ||
| include README.md | ||
| include LICENSE | ||
| include MANIFEST.in | ||
| docs/_static/img/logo-name-medium.png | ||
| docs/_static/img/graphblas-vs-igraph.png | ||
| docs/_static/img/graphblas-vs-networkx.png |
Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. It's a small thing, but the last letter in "graphblas" is a bit cropped out. I think it still works, though, so not a serious issue. MemberAuthor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Really? I don't think I see that. Can you take a screenshot of what you are seeing? Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more.
I checked Chrome and Safari, both look like this for me... Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. And now that I'm looking at it again, the last s in algorithms is also gone... Interesting. Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. I tried playing around with zoom to see if that would make a diff, but nope. MemberAuthor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. I see this now (on my macbook); it renders fine for me on Windows where I created it. Thanks for letting me know! MemberAuthor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Okay, I think the logo should be fixed now if you'd like to take a look to verify. Member There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. To recap testing on my end: looks good on MacOS and Windows, looks truncated on Ubuntu. |
