Skip to content

Commit

Permalink
chore(ci): enable throughput benchmarks for zk-pok
Browse files Browse the repository at this point in the history
  • Loading branch information
soonum committed Feb 18, 2025
1 parent 0f44ffd commit c7b0fe3
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 3 deletions.
14 changes: 12 additions & 2 deletions .github/workflows/benchmark_tfhe_zk_pok.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,14 @@ name: tfhe-zk-pok benchmarks

on:
workflow_dispatch:
inputs:
bench_type:
description: "Benchmarks type"
type: choice
default: latency
options:
- latency
- throughput
push:
branches:
- main
Expand All @@ -20,6 +28,7 @@ env:
SLACK_ICON: https://pbs.twimg.com/profile_images/1274014582265298945/OjBKP9kn_400x400.png
SLACK_USERNAME: ${{ secrets.BOT_USERNAME }}
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
BENCH_TYPE: ${{ inputs.bench_type || 'latency' }}

jobs:
should-run:
Expand Down Expand Up @@ -105,7 +114,7 @@ jobs:

- name: Run benchmarks
run: |
make bench_tfhe_zk_pok
make BENCH_TYPE=${{ env.BENCH_TYPE }} bench_tfhe_zk_pok
- name: Parse results
run: |
Expand All @@ -119,7 +128,8 @@ jobs:
--commit-date "${{ env.COMMIT_DATE }}" \
--bench-date "${{ env.BENCH_DATE }}" \
--walk-subdirs \
--name-suffix avx512
--name-suffix avx512 \
--bench-type ${{ env.BENCH_TYPE }}
- name: Upload parsed results artifact
uses: actions/upload-artifact@65c4c4a1ddee5b72f698fdd19549f0f0fb45cf08
Expand Down
2 changes: 1 addition & 1 deletion tfhe/benches/integer/zk_pke.rs
Original file line number Diff line number Diff line change
Expand Up @@ -337,7 +337,7 @@ fn pke_zk_verify(c: &mut Criterion, results_file: &Path) {
"{bench_name}::throughput::{param_name}_{bits}_bits_packed_{zk_load}_ZK{zk_vers:?}"
);
bench_id_verify_and_expand = format!(
"{bench_name}_and_expand::{param_name}_{bits}_bits_packed_{zk_load}_ZK{zk_vers:?}"
"{bench_name}_and_expand::throughput::{param_name}_{bits}_bits_packed_{zk_load}_ZK{zk_vers:?}"
);

println!("Generating proven ciphertexts list ({zk_load})... ");
Expand Down

0 comments on commit c7b0fe3

Please sign in to comment.