-
Notifications
You must be signed in to change notification settings - Fork 252
New Python tutorial module graphframes.tutorial #513
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
Conversation
…s.txt and split out requirements-dev.txt. Version bumps.
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.
This is here within the package, as is common in Java projects, to wholly encapsulate the tutorial within the project in a standard way. I don't want to rely on the user having this or that operating or filesystem.
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.
This is executable Click CLI to download any stackexchange.com
site, I think stackoverflow.com
would need a small change, the rest work.
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.
This file is to provide for a working motif tutorial on Github, where the motif finding tutorial I wrote in Markdown doesn't display correctly.
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.
This file builds a knowledge graph nodes / edge files out of the Stack Exchange data dump XML files. I plan to create a knowledge graph construction tutorial and this file is a Github readable version of that future Markdown tutorial.
I want to use the files this creates for extended unit tests with a more realistic workload.
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.
These are utilities that would distract from the code in the other files.
Closed in favor of a minimized PR #518. |
Closing in favor of minimized PR. |
This is a sub-PR of #473. It is the code that corresponds to #511.
These changes are necessary to:
graphframes.tutorials.donwload
graphframes.tutorials.stackexchange
graphframes.tutorials.stackexchange
graphframes.tutorials.motif
- this is here to provide for future unit testability of tutorials and as a Github browsable reference that matches the Motif Finding tutorial in 3 of 3: Documentation cleanup and update. Added a motif finding tutorial. #511.In addition:
Why are the changes needed?
These changes are needed to make the docs in #511 work. Otherwise that PR's new Motif Finding Tutorial won't work. Merge me first :)