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

significant reorganization #1

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

poldrack
Copy link

hi - I have gone in and done some pretty substanatial reorganization. would be great to merge this into the main repo, but I will understand if you don't want to. major features:

  • set up as a standard python package, with setup.py
  • moved example script that runs the analysis into scripts/run_node2vec.py
  • moved helper functions from that script into the main node2vec/node2vec.py
  • wrote a couple of simple tests (in the tests/ directory)

I tried getting it working in python3 but something funky is going on, as the main test hangs and never completes, so I added a test in the setup.py to ensure that the user is using python 2.7. Will try to fix that later.

I am planning on using this to do some analyses of neuroimaging data, after our discussion with Jure yesterday. will add in those functions as they develop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant