Skip to content

Commit 0956e1f

Browse files
committed
Fix some mismatches
1 parent a66af9b commit 0956e1f

File tree

5 files changed

+11
-56
lines changed

5 files changed

+11
-56
lines changed

src/e2e/mod.rs

Lines changed: 5 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -5,46 +5,25 @@ use crate::zkvm_verifier::binding::{
55
ZKVMProofInput, E, F,
66
};
77

8-
use crate::basefold_verifier::query_phase::QueryPhaseVerifierInput;
9-
use crate::zkvm_verifier::verifier::{verify_gkr_circuit, verify_zkvm_proof};
8+
use crate::zkvm_verifier::verifier::verify_zkvm_proof;
109
use ceno_mle::util::ceil_log2;
11-
use ceno_transcript::BasicTranscript;
1210
use ceno_zkvm::scheme::ZKVMProof;
1311
use ceno_zkvm::structs::ZKVMVerifyingKey;
1412
use ff_ext::BabyBearExt4;
15-
use gkr_iop::gkr::{
16-
layer::sumcheck_layer::{SumcheckLayer, SumcheckLayerProof},
17-
GKRCircuit,
18-
};
19-
use itertools::Itertools;
2013
use mpcs::{Basefold, BasefoldRSParams};
21-
use openvm_circuit::arch::{
22-
instructions::program::Program, verify_single, SystemConfig, VirtualMachine, VmExecutor,
23-
};
24-
use openvm_native_circuit::{Native, NativeConfig};
14+
use openvm_circuit::arch::instructions::program::Program;
2515
use openvm_native_compiler::{
2616
asm::AsmBuilder,
2717
conversion::{convert_program, CompilerOptions},
2818
prelude::AsmCompiler,
2919
};
3020
use openvm_native_recursion::hints::Hintable;
3121
use openvm_stark_backend::config::StarkGenericConfig;
32-
use openvm_stark_sdk::{
33-
config::{
34-
baby_bear_poseidon2::{BabyBearPoseidon2Config, BabyBearPoseidon2Engine},
35-
fri_params::standard_fri_params_with_100_bits_conjectured_security,
36-
setup_tracing_with_log_level, FriParameters,
37-
},
38-
engine::StarkFriEngine,
39-
p3_baby_bear::BabyBear,
40-
};
41-
use std::fs::File;
22+
use openvm_stark_sdk::config::baby_bear_poseidon2::BabyBearPoseidon2Config;
4223

4324
type SC = BabyBearPoseidon2Config;
4425
type EF = <SC as StarkGenericConfig>::Challenge;
4526

46-
use ceno_zkvm::{scheme::verifier::ZKVMVerifier, structs::ComposedConstrainSystem};
47-
4827
pub fn parse_zkvm_proof_import(
4928
zkvm_proof: ZKVMProof<BabyBearExt4, Basefold<BabyBearExt4, BasefoldRSParams>>,
5029
vk: &ZKVMVerifyingKey<BabyBearExt4, Basefold<BabyBearExt4, BasefoldRSParams>>,
@@ -262,7 +241,7 @@ pub fn parse_zkvm_proof_import(
262241
(
263242
1,
264243
SumcheckLayerProofInput {
265-
proof: iop_messages,
244+
proof: iop_messages.into(),
266245
evals,
267246
},
268247
)
@@ -290,7 +269,7 @@ pub fn parse_zkvm_proof_import(
290269
}
291270

292271
let main = SumcheckLayerProofInput {
293-
proof: iop_messages,
272+
proof: iop_messages.into(),
294273
evals,
295274
};
296275

src/tower_verifier/binding.rs

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ impl Hintable<InnerConfig> for IOPProverMessage {
157157
impl VecAutoHintable for IOPProverMessage {}
158158

159159
/// Assume that all the prover messages have the same size.
160-
#[derive(Debug, Deserialize)]
160+
#[derive(Debug, Deserialize, Default)]
161161
pub struct IOPProverMessageVec {
162162
pub prover_message_size: usize,
163163
pub data: Vec<E>,
@@ -296,20 +296,3 @@ impl Hintable<InnerConfig> for ThreeDimensionalVector {
296296
stream
297297
}
298298
}
299-
300-
pub struct TowerVerifierInput {
301-
pub prod_out_evals: Vec<Vec<E>>,
302-
pub logup_out_evals: Vec<Vec<E>>,
303-
pub num_variables: Vec<usize>,
304-
pub num_fanin: usize,
305-
306-
// TowerProof
307-
pub num_proofs: usize,
308-
pub num_prod_specs: usize,
309-
pub num_logup_specs: usize,
310-
pub _max_num_variables: usize,
311-
312-
pub proofs: Vec<Vec<IOPProverMessage>>,
313-
pub prod_specs_eval: Vec<Vec<Vec<E>>>,
314-
pub logup_specs_eval: Vec<Vec<Vec<E>>>,
315-
}

src/tower_verifier/program.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
use super::binding::{IOPProverMessageVariable, PointAndEvalVariable, PointVariable};
1+
use super::binding::{PointAndEvalVariable, PointVariable};
22
use crate::arithmetics::{
33
challenger_multi_observe, eq_eval, evaluate_at_point_degree_1, extend, exts_to_felts,
44
fixed_dot_product, reverse, UniPolyExtrapolator,

src/zkvm_verifier/binding.rs

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,12 @@ use crate::arithmetics::next_pow2_instance_padding;
22
use crate::basefold_verifier::basefold::{
33
BasefoldCommitment, BasefoldCommitmentVariable, BasefoldProof, BasefoldProofVariable,
44
};
5-
use crate::basefold_verifier::query_phase::{
6-
QueryPhaseVerifierInput, QueryPhaseVerifierInputVariable,
7-
};
5+
86
use crate::tower_verifier::binding::{
97
IOPProverMessageVec, IOPProverMessageVecVariable, ThreeDimensionalVecVariable,
108
ThreeDimensionalVector,
119
};
12-
use crate::{
13-
arithmetics::ceil_log2,
14-
tower_verifier::binding::{IOPProverMessage, IOPProverMessageVariable, PointVariable},
15-
};
10+
use crate::{arithmetics::ceil_log2, tower_verifier::binding::PointVariable};
1611
use itertools::Itertools;
1712
use openvm_native_compiler::{
1813
asm::AsmConfig,
@@ -371,7 +366,7 @@ impl Hintable<InnerConfig> for ZKVMChipProofInput {
371366

372367
#[derive(Default)]
373368
pub struct SumcheckLayerProofInput {
374-
pub proof: Vec<IOPProverMessage>,
369+
pub proof: IOPProverMessageVec,
375370
pub evals: Vec<E>,
376371
}
377372
#[derive(DslVariable, Clone)]

src/zkvm_verifier/verifier.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,7 @@ use ceno_mle::StructuralWitInType::{
3636
StackedConstantSequence, StackedIncrementalSequence,
3737
};
3838
use ceno_zkvm::structs::{VerifyingKey, ZKVMVerifyingKey};
39-
use ceno_zkvm::{
40-
circuit_builder::SetTableSpec, scheme::verifier::ZKVMVerifier, structs::ComposedConstrainSystem,
41-
};
39+
use ceno_zkvm::{circuit_builder::SetTableSpec, structs::ComposedConstrainSystem};
4240
use ff_ext::BabyBearExt4;
4341
use gkr_iop::gkr::layer::ROTATION_OPENING_COUNT;
4442
use gkr_iop::{

0 commit comments

Comments
 (0)