Skip to content

Generic Merkle Channel in Prover#466

Merged
Gali-StarkWare merged 1 commit intomainfrom
gali/generic_merkle_channel_prover
Apr 28, 2026
Merged

Generic Merkle Channel in Prover#466
Gali-StarkWare merged 1 commit intomainfrom
gali/generic_merkle_channel_prover

Conversation

@Gali-StarkWare
Copy link
Copy Markdown
Contributor

No description provided.

@reviewable-StarkWare
Copy link
Copy Markdown
Collaborator

This change is Reviewable

Copy link
Copy Markdown
Contributor Author

Gali-StarkWare commented Apr 27, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Gali-StarkWare Gali-StarkWare mentioned this pull request Apr 27, 2026
@Gali-StarkWare Gali-StarkWare marked this pull request as ready for review April 27, 2026 14:34
@Gali-StarkWare Gali-StarkWare self-assigned this Apr 27, 2026
@ilyalesokhin-starkware
Copy link
Copy Markdown
Contributor

crates/circuit_prover/src/prover.rs line 42 at r1 (raw file):

const COMPOSITION_POLYNOMIAL_LOG_DEGREE_BOUND: u32 = 1;

pub struct CircuitProof<H: MerkleHasherLifted = Blake2sM31MerkleHasher> {

What happens if you don't give a default?

Suggestion:

<H: MerkleHasherLifted>

Copy link
Copy Markdown
Contributor

@ilyalesokhin-starkware ilyalesokhin-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ilyalesokhin-starkware reviewed 2 files and all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on Gali-StarkWare).

Copy link
Copy Markdown
Contributor Author

@Gali-StarkWare Gali-StarkWare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Gali-StarkWare made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on ilyalesokhin-starkware).


crates/circuit_prover/src/prover.rs line 42 at r1 (raw file):

Previously, ilyalesokhin-starkware wrote…

What happens if you don't give a default?

Done.

@Gali-StarkWare Gali-StarkWare force-pushed the gali/generic_merkle_channel_prover branch from dcf2bf5 to a8f3bd2 Compare April 27, 2026 15:21
Copy link
Copy Markdown
Contributor

@ilyalesokhin-starkware ilyalesokhin-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@ilyalesokhin-starkware reviewed 3 files and all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on Gali-StarkWare).

@Gali-StarkWare Gali-StarkWare merged commit 611b94f into main Apr 28, 2026
11 checks passed
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.

3 participants