Skip to content
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

Bolt #97

Merged
merged 6 commits into from Oct 9, 2018

Conversation

Projects
None yet
2 participants
@cwharris
Copy link
Contributor

commented Oct 5, 2018

  • .gitignore: added pycharm, vscode related directories
  • bolt / cypher support + jupyter notebook demo
@cwharris

This comment has been minimized.

Copy link
Contributor Author

commented Oct 5, 2018

I still need to run through this to clean up.

@lmeyerov i know we talked about performance / choking while converting data, but Bolt’s graph’s relationships has a structure such as {is:, from:, to:, props:{...}}, and I’ve not figured out how to view those props in the graph viewer without collapsing them to a single dictionary.

@cwharris cwharris requested a review from lmeyerov Oct 5, 2018

@cwharris cwharris force-pushed the cwharris:bolt branch 2 times, most recently from 439abb3 to 05352ed Oct 5, 2018

@cwharris cwharris force-pushed the cwharris:bolt branch from 05352ed to 29920fd Oct 5, 2018

@cwharris

This comment has been minimized.

Copy link
Contributor Author

commented Oct 5, 2018

Ready for review.

Show resolved Hide resolved demos/Neo4jBoltCypher.ipynb Outdated
Show resolved Hide resolved graphistry/plotter.py Outdated
Show resolved Hide resolved graphistry/plotter.py Outdated
Show resolved Hide resolved graphistry/plotter.py Outdated
Show resolved Hide resolved graphistry/plotter.py Outdated
Show resolved Hide resolved graphistry/pygraphistry.py Outdated

Christopher Harris added some commits Oct 8, 2018

Show resolved Hide resolved graphistry/bolt_util.py Outdated
Show resolved Hide resolved graphistry/plotter.py Outdated
@lmeyerov

This comment has been minimized.

Copy link
Contributor

commented Oct 8, 2018

@cwharris Just made another pass, about there. mostly help(grapbhistry...) and testing what happens if no nodes in return.

@cwharris cwharris merged commit 4ef8491 into graphistry:master Oct 9, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@cwharris cwharris deleted the cwharris:bolt branch Oct 9, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.