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.

Sign up for GitHub

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

Jump to bottom

Weighted edges plugin #73

Closed
moaxcp opened this issue May 8, 2017 · 2 comments
Closed

Weighted edges plugin #73

moaxcp opened this issue May 8, 2017 · 2 comments

Comments

@moaxcp
Copy link
Owner

moaxcp commented May 8, 2017

  • traverseEdges needs to return edges sorted by weight.
@moaxcp
Copy link
Owner Author

moaxcp commented May 17, 2017

Changing edges to a TreeSet would be better since all methods would return sorted results.

@moaxcp
Copy link
Owner Author

moaxcp commented May 17, 2017

Should add method to graph which tracks a list of traits to add to all current and future edges.

moaxcp added a commit that referenced this issue Jun 3, 2017
@moaxcp
Merge tag '0.15.0' into develop
25cf13f 
This release combines fixes for a few issues on github.

* [#75](#75) Vertex.name, Edge.one, and Edge.two is now @PackageScope. This only
affects code that is @CompileStatic for now due to [GROOVY-3010](https://issues.apache.org/jira/browse/GROOVY-3010).
* [#74](#74 and edges may now be deleted. A vertex cannot be deleted if
there are edges referencing it.
* [#73](#73) Added EdgeWeightPlugin. This plugin adds the Weight trait to each
edge. Traversal methods are ordered by edge weight.

There were also several other changes that were not an issue on github:

Updated gradle to 3.5. Refactored gradle script to use the plugins closure when possible. gradle-gitflow does not work
with the closure because it is not in the gradle repository. This is another reason to update the plugin. Spock was also
updated to 1.1.

Added edgeTraits and vertexTraits. These methods will ensure traits are applied to current and future edges and vertices.

Added tests to provide more code coverage in jacoco.

Added more usage details to README.md
@moaxcp moaxcp closed this as completed Jun 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@moaxcp