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

Implementation of Depth Limited Search #157 #164

Merged
merged 1 commit into from
Jan 7, 2016

Conversation

gabizekany
Copy link
Contributor

Updated DLS unit tests

Fix for Improve GraphBuilder to generate graphs without edge values #154

  • created function in GraphBuilder called connect(vertex , vertex ) which connects two vertices with an empty valued edge.

Add license agreement header

Remove unnecessary code

Fix test for GraphBuilder edge testing

Updated DLS unit tests

Fix for Improve GraphBuilder to generate graphs without edge values citiususc#154

- created function in GraphBuilder called connect(vertex , vertex ) which connects two vertices with an empty valued edge.

Add license agreement header

Remove unnecessary code

Fix test for GraphBuilder edge testing
@codecov-io
Copy link

Current coverage is 53.79%

Merging #164 into 1.0.X will increase coverage by +0.96% as of cbdf1d8

@@            1.0.X    #164   diff @@
=====================================
  Files          57      58     +1
  Stmts        1622    1714    +92
  Branches      264     274    +10
  Methods                         
=====================================
+ Hit           857     922    +65
- Partial        63      68     +5
- Missed        702     724    +22

Review entire Coverage Diff as of cbdf1d8

Powered by Codecov. Updated on successful CI builds.

@pablormier
Copy link
Contributor

Thanks for your contrib!. I think most of these changes will be merged into the DFS in a future, since it's a particular case of the DFS

pablormier added a commit that referenced this pull request Jan 7, 2016
Implementation of Depth Limited Search #157
@pablormier pablormier merged commit 5b26630 into citiususc:1.0.X Jan 7, 2016
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.

3 participants