Skip to content

Update README.md#36

Open
jpfairbanks wants to merge 2 commits intomasterfrom
jpfairbanks-patch-2
Open

Update README.md#36
jpfairbanks wants to merge 2 commits intomasterfrom
jpfairbanks-patch-2

Conversation

@jpfairbanks
Copy link
Copy Markdown
Collaborator

add install instructions

add install instructions
@rohitvarkey
Copy link
Copy Markdown
Collaborator

Now that JuliaLang/METADATA.jl#11174 is merged,Pkg.add should work.

add usage example
@jpfairbanks
Copy link
Copy Markdown
Collaborator Author

jpfairbanks commented Sep 14, 2017

I updated it, I think there might be something wrong with the foralledges function. Is the neighbor field being initialized correctly? See the example in this commit.

@rohitvarkey
Copy link
Copy Markdown
Collaborator

The edge from foralledges has the first 2 bits set to directions. edgeparse on the edge will give the correct neighbor value. This example in the docs shows how to use it.

Maybe I should roll this into foralledges to prevent the confusion?

@jpfairbanks
Copy link
Copy Markdown
Collaborator Author

A docs change will be sufficient. I totally forgot about this step when making the example.

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.

2 participants