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

Dijkstra maintenance #179

Merged
merged 25 commits into from
Dec 2, 2018
Merged

Dijkstra maintenance #179

merged 25 commits into from
Dec 2, 2018

Conversation

jGaboardi
Copy link
Member

Addressing #178

Minor function maintenance to the dijkstra algorithm.

@jGaboardi
swapping intial_distance variable name in dijkstra
124eec0
@jGaboardi
swapping float(inf) for np.inf in dijkstra
0d654bb
@jGaboardi
clean up inline doc
e15753b
@jGaboardi
fixing variable typo in dijkstra
8d5f1d3
@jGaboardi
altering unvisited set name in dijkstra
819b09b
@jGaboardi
updating unvisited set name in dijkstra
2500fba
@jGaboardi
updating variables in dijkstra
7d1987a
@jGaboardi
updating initial_dist docstring
3e07547
@jGaboardi
removing cost kwarg
6c1db97
@jGaboardi
updating cost kwarg example
d0e895d
@jGaboardi
updating cost kwarg docstring
9d731f5
@jGaboardi
removing cost kwarg
641f105
@jGaboardi
removing cost kwarg from network.node_neighbor_dist
f2a1b95
@jGaboardi
removing cost kwarg from single core proc.
4d1d9d1
@jGaboardi
removing cost kwarg from dijkstra multiprocessing
0a9efdd
@jGaboardi
updating cost kwarg docstring from dijkstra multiprocessing
128af15
@jGaboardi
updating cost kwarg example from dijkstra multiprocessing
74b43c0
@jGaboardi
removing cost kwarg from dijkstra multiprocessing
9c7d958
@jGaboardi
updating test_generate_tree in test_network.api
036e843
@jGaboardi
updating test_dijkstra in test_network.api
9824238
@jGaboardi
updating test_dijkstra_mp in test_network.api
1a51058
@jGaboardi
adding spacing for network.node_distance_matrix()
7951575
@jGaboardi jGaboardi self-assigned this Dec 2, 2018
@jGaboardi jGaboardi mentioned this pull request Dec 2, 2018
Closed
2 tasks
@coveralls
Copy link

Pull Request Test Coverage Report for Build 1021

  • 21 of 21 (100.0%) changed or added relevant lines in 3 files are covered.
  • 123 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.004%) to 98.22%
Files with Coverage Reduction New Missed Lines %
spaghetti/network.py 17 96.21%
spaghetti/util.py 26 100.0%
spaghetti/tests/test_network.py 40 99.64%
spaghetti/tests/test_network_api.py 40 99.57%
Totals Coverage Status
Change from base Build 1016: -0.004%
Covered Lines: 1324
Relevant Lines: 1348
💛 - Coveralls

@coveralls
Copy link

coveralls commented Dec 2, 2018 •
edited

Pull Request Test Coverage Report for Build 1022

  • 21 of 21 (100.0%) changed or added relevant lines in 3 files are covered.
  • 123 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.004%) to 98.22%
Files with Coverage Reduction New Missed Lines %
spaghetti/network.py 17 96.21%
spaghetti/util.py 26 100.0%
spaghetti/tests/test_network.py 40 99.64%
spaghetti/tests/test_network_api.py 40 99.57%
Totals Coverage Status
Change from base Build 1016: -0.004%
Covered Lines: 1324
Relevant Lines: 1348
💛 - Coveralls

@jGaboardi jGaboardi merged commit 59f0bd6 into pysal:master Dec 2, 2018
@jGaboardi jGaboardi deleted the dijk_maint branch December 2, 2018 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jGaboardi jGaboardi

Labels
package maintenance
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jGaboardi @coveralls