Skip to content

Commit

Permalink
move gob to stdlib; test passes w merge rec ver
Browse files Browse the repository at this point in the history
  • Loading branch information
ledwards2225 committed Dec 6, 2023
1 parent 228f4f3 commit 3c27245
Show file tree
Hide file tree
Showing 9 changed files with 11 additions and 81 deletions.
1 change: 0 additions & 1 deletion barretenberg/cpp/src/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ add_subdirectory(barretenberg/env)
add_subdirectory(barretenberg/examples)
add_subdirectory(barretenberg/flavor)
add_subdirectory(barretenberg/grumpkin_srs_gen)
add_subdirectory(barretenberg/goblin)
add_subdirectory(barretenberg/honk)
add_subdirectory(barretenberg/join_split_example)
add_subdirectory(barretenberg/numeric)
Expand Down
2 changes: 1 addition & 1 deletion barretenberg/cpp/src/barretenberg/eccvm/eccvm_prover.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@
#include "barretenberg/commitment_schemes/gemini/gemini.hpp"
#include "barretenberg/commitment_schemes/shplonk/shplonk.hpp"
#include "barretenberg/flavor/ecc_vm.hpp"
#include "barretenberg/goblin/translation_evaluations.hpp"
#include "barretenberg/plonk/proof_system/types/proof.hpp"
#include "barretenberg/relations/relation_parameters.hpp"
#include "barretenberg/sumcheck/sumcheck_output.hpp"
#include "barretenberg/transcript/transcript.hpp"
#include "barretenberg/translator_vm/translation_evaluations.hpp"

namespace proof_system::honk {

Expand Down
1 change: 0 additions & 1 deletion barretenberg/cpp/src/barretenberg/goblin/CMakeLists.txt

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
#include "barretenberg/eccvm/eccvm_composer.hpp"
#include "barretenberg/goblin/goblin.hpp"
#include "barretenberg/goblin/mock_circuits.hpp"
#include "barretenberg/goblin/translation_evaluations.hpp"
#include "barretenberg/proof_system/circuit_builder/eccvm/eccvm_circuit_builder.hpp"
#include "barretenberg/proof_system/circuit_builder/goblin_ultra_circuit_builder.hpp"
#include "barretenberg/proof_system/circuit_builder/ultra_circuit_builder.hpp"
#include "barretenberg/stdlib/recursion/goblin/goblin.hpp"
#include "barretenberg/stdlib/recursion/goblin/mock_circuits.hpp"
#include "barretenberg/stdlib/recursion/honk/verifier/ultra_recursive_verifier.hpp"
#include "barretenberg/translator_vm/goblin_translator_composer.hpp"
#include "barretenberg/translator_vm/translation_evaluations.hpp"
#include "barretenberg/ultra_honk/ultra_composer.hpp"

#include <gtest/gtest.h>
Expand Down Expand Up @@ -40,7 +40,8 @@ class GoblinRecursionTests : public ::testing::Test {
using NativeVerificationKey = flavor::GoblinUltra::VerificationKey;
using RecursiveFlavor = flavor::GoblinUltraRecursive_<GoblinUltraBuilder>;
using RecursiveVerifier = proof_system::plonk::stdlib::recursion::honk::UltraRecursiveVerifier_<RecursiveFlavor>;
using KernelInput = Goblin::AccumulationOutput;
using Goblin = proof_system::plonk::stdlib::recursion::goblin::Goblin;
using KernelInput = proof_system::plonk::stdlib::recursion::goblin::Goblin::AccumulationOutput;

/**
* @brief Construct a mock kernel circuit
Expand Down Expand Up @@ -72,7 +73,7 @@ class GoblinRecursionTests : public ::testing::Test {
*/
TEST_F(GoblinRecursionTests, Pseudo)
{
barretenberg::Goblin goblin;
Goblin goblin;

// Construct an initial circuit; its proof will be recursively verified by the first kernel
GoblinUltraBuilder initial_circuit{ goblin.op_queue };
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
#include "barretenberg/translator_vm/goblin_translator_composer.hpp"
#include "barretenberg/ultra_honk/ultra_composer.hpp"

namespace barretenberg {
namespace proof_system::plonk::stdlib::recursion::goblin {

class Goblin {
using HonkProof = proof_system::plonk::proof;
Expand Down Expand Up @@ -68,7 +68,6 @@ class Goblin {
// issue (https://github.com/AztecProtocol/barretenberg/issues/723)
if (verify_merge) {
RecursiveMergeVerifier merge_verifier{ &circuit_builder };
// Recursively verify the merge proof constructed on the previous call to accumulate
[[maybe_unused]] auto pairing_points = merge_verifier.verify_proof(merge_proof);
}

Expand Down Expand Up @@ -115,4 +114,4 @@ class Goblin {
return eccvm_verified && accumulator_construction_verified && translation_verified;
};
};
} // namespace barretenberg
} // namespace proof_system::plonk::stdlib::recursion::goblin
Original file line number Diff line number Diff line change
@@ -1,11 +1,10 @@
#include "barretenberg/commitment_schemes/commitment_key.hpp"
#include "barretenberg/eccvm/eccvm_composer.hpp"
#include "barretenberg/goblin/goblin.hpp"
#include "barretenberg/goblin/translation_evaluations.hpp"
#include "barretenberg/proof_system/circuit_builder/eccvm/eccvm_circuit_builder.hpp"
#include "barretenberg/proof_system/circuit_builder/goblin_ultra_circuit_builder.hpp"
#include "barretenberg/proof_system/circuit_builder/ultra_circuit_builder.hpp"
#include "barretenberg/translator_vm/goblin_translator_composer.hpp"
#include "barretenberg/translator_vm/translation_evaluations.hpp"
#include "barretenberg/ultra_honk/ultra_composer.hpp"

namespace barretenberg {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#pragma once
#include "barretenberg/flavor/goblin_translator.hpp"
#include "barretenberg/goblin/translation_evaluations.hpp"
#include "barretenberg/plonk/proof_system/types/proof.hpp"
#include "barretenberg/translator_vm/translation_evaluations.hpp"

namespace proof_system::honk {
class GoblinTranslatorVerifier {
Expand Down

0 comments on commit 3c27245

Please sign in to comment.