Skip to content

Conversation

@nsarlin-zama
Copy link
Contributor

@nsarlin-zama nsarlin-zama commented Oct 20, 2025

PR content/description

Since the zk verification does not perform any pbs, the usual heuristic for throuhput elements count does not work. We use a different heuristic:

For verify_and_expand, we use the classical pbs count and we take the lowest value between both heuristics.

I also removed the zkv1 integer benches. This reduces a lot the bench time, allows to reduce clutter in the grafana dashboard and we bench zkv1 in the tfhe-zk-pok crates anyways

@cla-bot cla-bot bot added the cla-signed label Oct 20, 2025
@nsarlin-zama nsarlin-zama force-pushed the ns/chore/fix_zk_thrpt_elements branch 2 times, most recently from cc50d4a to 7dde7b8 Compare October 21, 2025 12:39
@nsarlin-zama nsarlin-zama force-pushed the ns/chore/fix_zk_thrpt_elements branch from 7dde7b8 to a11e61c Compare October 31, 2025 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant