Skip to content

Dijkstra code simplification #2521

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
Merged
merged 30 commits into from
May 29, 2023
Merged

Dijkstra code simplification #2521

merged 30 commits into from
May 29, 2023

Conversation

@cvvergara
Copy link
Member

@cvvergara cvvergara commented May 25, 2023

Changes proposed in this pull request:

  • code simplification of dijkstra

@pgRouting/admins

cvvergara added 28 commits May 25, 2023 13:36
@cvvergara
[dijkstra][C++] pgr_dijkstra.hpp -> dijkstra.hpp
e137cd3
@cvvergara
[dijkstra] adjusting code to previous change
dd55df8
@cvvergara
[Dijkstra][visitors] creating a visitors file for dijkstra
95b20d7
@cvvergara
[dijkstra] adjusting code to previous change
61cf7ac
@cvvergara
[Dijkstra][visitors] moving visitors got visitors file
51f7850
@cvvergara
[dijkstra] adjusting code to previous change
80ef5dc
@cvvergara
[dijkstra]:broom: removing unused code & linting
8f38334
@cvvergara
[Dijkstra][visitors] moving visitor to visitors file
ab11cf1
@cvvergara
[dijkstra] adjusting code to previous change
fc1118f
@cvvergara
[dijkstra]:broom: removing unused code
7182b7f
@cvvergara
[dijkstra] 1 to 1 is a function
92f0efb
@cvvergara
[dijkstra]:broom: removing unused code
97cdb61
@cvvergara
[dijkstra] making call with map a function
2721232
@cvvergara
[dijkstra] adjusting code to previous change
40f338f
@cvvergara
[dijkstra]:broom: removing unused code
180f666
@cvvergara
[dijkstra] many to many not used anymore
face553
@cvvergara
[dijkstra] marking unused code
37cc640
@cvvergara
[dijkstra]:broom: removing unused code
bd0ca0b
@cvvergara
Separating driving distance from dijkstra
ffe352d
@cvvergara
[dijkstra/drivingDistance] adjusting code to previous change
a4303cd
@cvvergara
[dijkstra/drivingDistance]:broom: removing unused code
1315106
@cvvergara
dijkstra is on algorithms namespace
e3bc881
@cvvergara
[dijkstra] Using departure & destination when data has boost graph info
afb189a
@cvvergara
[1 to many] Using Boost graph in fn that calls boost
9b9915d
@cvvergara
[dijkstra] 1 to 1 using combinations
d2d776d
@cvvergara
[dijkstra]:broom: removing unused code
48c596a
@cvvergara
fixing licenses
bb928f4
@cvvergara
[docqueries] Removing windows ctrl-M
9f9f6d7
@cvvergara cvvergara added this to the Release 3.6.0 milestone May 25, 2023
@cvvergara cvvergara requested review from krashish8 and robe2 May 25, 2023 20:02
@cvvergara cvvergara marked this pull request as draft May 25, 2023 20:05
@cvvergara cvvergara marked this pull request as ready for review May 25, 2023 20:53
@cvvergara cvvergara merged commit 7586bbf into pgRouting:develop May 29, 2023
@cvvergara cvvergara deleted the dijkstra-code-simplification branch May 29, 2023 14:29
@cvvergara cvvergara mentioned this pull request May 29, 2023
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@robe2 robe2 robe2 approved these changes

@krashish8 krashish8 krashish8 approved these changes

Assignees

No one assigned

Labels

C/C++ Enhancement

Projects

None yet

Milestone

Release 3.6.0

Development

Successfully merging this pull request may close these issues.

3 participants

@cvvergara @robe2 @krashish8