Skip to content

Commit 08c566c

Browse files
authored
Merge pull request #3103 from TheBlueMatt/2024-06-real-bench
Use a real (probing-generated) scorer in benchmarks
2 parents 6773266 + 7ae311d commit 08c566c

File tree

4 files changed

+161
-269
lines changed

4 files changed

+161
-269
lines changed

.github/workflows/build.yml

Lines changed: 23 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -83,19 +83,36 @@ jobs:
8383
id: cache-graph
8484
uses: actions/cache@v3
8585
with:
86-
path: lightning/net_graph-2023-01-18.bin
87-
key: ldk-net_graph-v0.0.113-2023-01-18.bin
86+
path: lightning/net_graph-2023-12-10.bin
87+
key: ldk-net_graph-v0.0.118-2023-12-10.bin
8888
- name: Fetch routing graph snapshot
8989
if: steps.cache-graph.outputs.cache-hit != 'true'
9090
run: |
91-
curl --verbose -L -o lightning/net_graph-2023-01-18.bin https://bitcoin.ninja/ldk-net_graph-v0.0.113-2023-01-18.bin
92-
echo "Sha sum: $(sha256sum lightning/net_graph-2023-01-18.bin | awk '{ print $1 }')"
93-
if [ "$(sha256sum lightning/net_graph-2023-01-18.bin | awk '{ print $1 }')" != "${EXPECTED_ROUTING_GRAPH_SNAPSHOT_SHASUM}" ]; then
91+
curl --verbose -L -o lightning/net_graph-2023-12-10.bin https://bitcoin.ninja/ldk-net_graph-v0.0.118-2023-12-10.bin
92+
echo "Sha sum: $(sha256sum lightning/net_graph-2023-12-10.bin | awk '{ print $1 }')"
93+
if [ "$(sha256sum lightning/net_graph-2023-12-10.bin | awk '{ print $1 }')" != "${EXPECTED_ROUTING_GRAPH_SNAPSHOT_SHASUM}" ]; then
9494
echo "Bad hash"
9595
exit 1
9696
fi
9797
env:
98-
EXPECTED_ROUTING_GRAPH_SNAPSHOT_SHASUM: da6066f2bddcddbe7d8a6debbd53545697137b310bbb8c4911bc8c81fc5ff48c
98+
EXPECTED_ROUTING_GRAPH_SNAPSHOT_SHASUM: e94b38ef4b3ce683893bf6a3ee28d60cb37c73b059403ff77b7e7458157968c2
99+
- name: Cache scorer snapshot
100+
id: cache-scorer
101+
uses: actions/cache@v3
102+
with:
103+
path: lightning/scorer-2023-12-10.bin
104+
key: ldk-scorer-v0.0.118-2023-12-10.bin
105+
- name: Fetch scorer snapshot
106+
if: steps.cache-scorer.outputs.cache-hit != 'true'
107+
run: |
108+
curl --verbose -L -o lightning/scorer-2023-12-10.bin https://bitcoin.ninja/ldk-scorer-v0.0.118-2023-12-10.bin
109+
echo "Sha sum: $(sha256sum lightning/scorer-2023-12-10.bin | awk '{ print $1 }')"
110+
if [ "$(sha256sum lightning/scorer-2023-12-10.bin | awk '{ print $1 }')" != "${EXPECTED_SCORER_SNAPSHOT_SHASUM}" ]; then
111+
echo "Bad hash"
112+
exit 1
113+
fi
114+
env:
115+
EXPECTED_SCORER_SNAPSHOT_SHASUM: 570a26bb28870fe1da7e392cdec9fb794718826b04c43ca053d71a8a9bb9be69
99116
- name: Fetch rapid graph sync reference input
100117
run: |
101118
curl --verbose -L -o lightning-rapid-gossip-sync/res/full_graph.lngossip https://bitcoin.ninja/ldk-compressed_graph-285cb27df79-2022-07-21.bin

lightning/src/routing/gossip.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3746,7 +3746,7 @@ pub mod benches {
37463746

37473747
pub fn read_network_graph(bench: &mut Criterion) {
37483748
let logger = crate::util::test_utils::TestLogger::new();
3749-
let mut d = crate::routing::router::bench_utils::get_route_file().unwrap();
3749+
let (mut d, _) = crate::routing::router::bench_utils::get_graph_scorer_file().unwrap();
37503750
let mut v = Vec::new();
37513751
d.read_to_end(&mut v).unwrap();
37523752
bench.bench_function("read_network_graph", |b| b.iter(||
@@ -3756,7 +3756,7 @@ pub mod benches {
37563756

37573757
pub fn write_network_graph(bench: &mut Criterion) {
37583758
let logger = crate::util::test_utils::TestLogger::new();
3759-
let mut d = crate::routing::router::bench_utils::get_route_file().unwrap();
3759+
let (mut d, _) = crate::routing::router::bench_utils::get_graph_scorer_file().unwrap();
37603760
let net_graph = NetworkGraph::read(&mut d, &logger).unwrap();
37613761
bench.bench_function("write_network_graph", |b| b.iter(||
37623762
black_box(&net_graph).encode()

0 commit comments

Comments
 (0)