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

Replace boost::hash by std::hash #6892

Merged
merged 3 commits into from
May 23, 2024
Merged

Replace boost::hash by std::hash #6892

merged 3 commits into from
May 23, 2024

Conversation

DennisOSRM
Copy link
Collaborator

@DennisOSRM DennisOSRM commented May 23, 2024

Benchmark Results

Benchmark Base PR
alias aliased u32: 1144.85
plain u32: 1147.73
aliased double: 1203.45
plain double: 1206.12
aliased u32: 1145.06
plain u32: 1147.12
aliased double: 1192.15
plain double: 1221.44
json-render String: 8.54592ms
Stringstream: 11.5546ms
Vector: 7.5724ms
String: 8.55746ms
Stringstream: 11.7931ms
Vector: 7.65687ms
match_ch Default radius:
4.48136ms/req at 82 coordinate
0.0546507ms/coordinate
Radius 5m:
4.47619ms/req at 82 coordinate
0.0545877ms/coordinate
Radius 10m:
15.259ms/req at 82 coordinate
0.186085ms/coordinate
Radius 15m:
36.8768ms/req at 82 coordinate
0.449718ms/coordinate
Radius 30m:
313.326ms/req at 82 coordinate
3.82104ms/coordinate
Default radius:
4.46941ms/req at 82 coordinate
0.0545049ms/coordinate
Radius 5m:
4.45022ms/req at 82 coordinate
0.054271ms/coordinate
Radius 10m:
15.1737ms/req at 82 coordinate
0.185045ms/coordinate
Radius 15m:
36.9263ms/req at 82 coordinate
0.450321ms/coordinate
Radius 30m:
313.703ms/req at 82 coordinate
3.82564ms/coordinate
match_mld Default radius:
4.02955ms/req at 82 coordinate
0.0491409ms/coordinate
Radius 5m:
3.33361ms/req at 82 coordinate
0.0406537ms/coordinate
Radius 10m:
12.1625ms/req at 82 coordinate
0.148324ms/coordinate
Radius 15m:
31.4254ms/req at 82 coordinate
0.383237ms/coordinate
Radius 30m:
350.356ms/req at 82 coordinate
4.27263ms/coordinate
Default radius:
3.3591ms/req at 82 coordinate
0.0409647ms/coordinate
Radius 5m:
3.34183ms/req at 82 coordinate
0.0407541ms/coordinate
Radius 10m:
12.2526ms/req at 82 coordinate
0.149422ms/coordinate
Radius 15m:
31.8058ms/req at 82 coordinate
0.387876ms/coordinate
Radius 30m:
348.725ms/req at 82 coordinate
4.25274ms/coordinate
packedvector random write:
std::vector 10841.6 ms
util::packed_vector 74460.6 ms
slowdown: 6.86805
random read:
std::vector 8816.26 ms
util::packed_vector 30924.7 ms
slowdown: 3.50769
random write:
std::vector 11452.8 ms
util::packed_vector 82938.7 ms
slowdown: 7.2418
random read:
std::vector 11367.9 ms
util::packed_vector 33705.6 ms
slowdown: 2.96497
rtree 1 result:
203.53ms -> 0.020353 ms/query
10 results:
236.981ms -> 0.0236981 ms/query
1 result:
206.692ms -> 0.0206692 ms/query
10 results:
242.83ms -> 0.024283 ms/query

@DennisOSRM DennisOSRM merged commit 46dc660 into master May 23, 2024
21 checks passed
@DennisOSRM DennisOSRM deleted the retire_boost_hash branch May 23, 2024 12:35
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