Skip to content

feat: Loading setup for proving network#100

Merged
AlexOnTheStorm merged 12 commits intomainfrom
ad_proof_compressor_service
Jun 4, 2025
Merged

feat: Loading setup for proving network#100
AlexOnTheStorm merged 12 commits intomainfrom
ad_proof_compressor_service

Conversation

@AlexOnTheStorm
Copy link
Contributor

@AlexOnTheStorm AlexOnTheStorm commented May 21, 2025

What ❔

Proof compressor now provides convenient dyn trait setup interfaces, simple and sync, both for proving and generating keys and allows using in-memory cache for setup
Potential weak async places causing sporadic errors are gone e.g. async init_context
Improved performance. Before: 71s each proof; now: 71s first proof and 63s each following proof.

Why ❔

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.

@AlexOnTheStorm AlexOnTheStorm requested a review from EmilLuta May 21, 2025 15:42
@AlexOnTheStorm AlexOnTheStorm requested a review from robik75 June 4, 2025 11:02
robik75
robik75 previously approved these changes Jun 4, 2025
@Artemka374
Copy link
Contributor

Although I don't want my comments to be a blocker to your work. But please, consider to probably do a follow-up when you have some free time and move out of panics(at least in places you touch)

@AlexOnTheStorm AlexOnTheStorm merged commit 31b4967 into main Jun 4, 2025
17 checks passed
@AlexOnTheStorm AlexOnTheStorm deleted the ad_proof_compressor_service branch June 4, 2025 12:15
AlexOnTheStorm pushed a commit that referenced this pull request Jun 4, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.154.3](v0.154.2...v0.154.3)
(2025-06-04)


### Features

* Loading setup for proving network
([#100](#100))
([31b4967](31b4967))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: zksync-era-bot <zksync-era-bot@users.noreply.github.com>
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.

4 participants