Skip to content

Commit

Permalink
WIP: temporarily disable key-verification constraints
Browse files Browse the repository at this point in the history
  • Loading branch information
dtebbs committed Sep 2, 2020
1 parent cde20b5 commit 2c1c259
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
4 changes: 2 additions & 2 deletions libzecale/circuits/aggregator_circuit_wrapper.tcc
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ aggregator_circuit_wrapper<wppT, wsnarkT, nverifierT, hashT, NumProofs>::
for (size_t i = 0; i < NumProofs; ++i) {
_nested_proofs[i]->generate_r1cs_constraints();
}
_nested_vk_hash_gadget->generate_r1cs_constraints();
// _nested_vk_hash_gadget->generate_r1cs_constraints();
_aggregator_gadget->generate_r1cs_constraints();
}

Expand Down Expand Up @@ -157,7 +157,7 @@ libzeth::extended_proof<wppT, wsnarkT> aggregator_circuit_wrapper<
_nested_vk->generate_r1cs_witness(nested_vk);

// Witness hash of verification keypair
_nested_vk_hash_gadget->generate_r1cs_witness();
// _nested_vk_hash_gadget->generate_r1cs_witness();

// Pass the input values (in npp) to the aggregator gadget.
_aggregator_gadget->generate_r1cs_witness(nested_inputs);
Expand Down
9 changes: 5 additions & 4 deletions libzecale/tests/aggregator/aggregator_dummy_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,11 @@ void test_aggregator_with_batch(
size_t winput_idx = 0;

// Check the nested vk hash
libff::Fr<wppT> expect_nested_vk_hash =
verification_key_hash_gadget<wppT, nverifierT, hash<wppT>>::
compute_hash(nkp.vk, num_inputs_per_nested_proof);
ASSERT_EQ(expect_nested_vk_hash, winputs[winput_idx++]);
// libff::Fr<wppT> expect_nested_vk_hash =
// verification_key_hash_gadget<wppT, nverifierT, hash<wppT>>::
// compute_hash(nkp.vk, num_inputs_per_nested_proof);
// ASSERT_EQ(expect_nested_vk_hash, winputs[winput_idx++]);
++winput_idx;

for (size_t proof_idx = 0; proof_idx < batch_size; ++proof_idx) {
// Check that each input from the batch appears as expected in the
Expand Down

0 comments on commit 2c1c259

Please sign in to comment.