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

Tutorial needs to be clear that v2.2 needs to be used, or brought up-to-date with graphlab-code. #105

Open
hoytak opened this issue Dec 2, 2013 · 2 comments

Comments

@hoytak
Copy link

hoytak commented Dec 2, 2013

The current tutorial doesn't work off of the master branch of the github repository (found trying to coach a friend through the installation and starting examples.

@dcpjoncas
Copy link

Noticed two small problems with the tutorial:

  1. use of graphlab::distributed_graph_type as opposed to graphlab::distributed_graph. This is correct in most of the tutorial except on page http://docs.graphlab.org/using_graphlab_distributed_graph_load_data.html following "At this point, your code will look like this:".

  2. Both instances of the line parser have bugs in them on page http://docs.graphlab.org/using_graphlab_distributed_graph_load_data.html again.

The first version has "retun true" instead of "return true", while the version following "At this point, your code will look like this:" is missing the "return true" entirely.

@boorad
Copy link
Contributor

boorad commented Feb 2, 2014

Oops, I didn't see this issue here, and filed #122 . On the plus-side, #122 has a patch / pull request...

boorad added a commit to boorad/graphlab that referenced this issue Feb 2, 2014
another change to accompany jegonzal#122 and jegonzal#105
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

No branches or pull requests

3 participants