Skip to content

Commit

Permalink
refactor: Join-split example Part 1 (#4965)
Browse files Browse the repository at this point in the history
The first of a two-part series to refactor the join-split example, the
goal being to focus the tests on circuit writing (no proof
construction), update them to reflect our newer idioms, and relocate
them. Key changes
1) Keep a subset of tests relating to circuit writing.
1) Remove code that has to do with proof construction.
1) Add a new circuit change detection paradigm: rather than hashing a
verification key (a proving system-dependent object), simply hash the
circuit-defining data in the circuit builder.
1) Delete unused code (some unused before, some unused now). 

The next PR AztecProtocol/aztec-packages#5016
will move and condense files, and update namespaces.
  • Loading branch information
codygunton authored and AztecBot committed Mar 9, 2024
1 parent c21e247 commit 82e2797
Show file tree
Hide file tree
Showing 26 changed files with 122 additions and 1,161 deletions.
3 changes: 3 additions & 0 deletions barretenberg.code-workspace
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,9 @@
// Some settings can only be configured here.
// The following are just provided as example.
"settings": {
"files.associations": {
"*.tcc": "cpp",
},
//
// Clangd. Note that this setting may be overridden by user settings
// to the default value "clangd".
Expand Down
1 change: 0 additions & 1 deletion cpp/scripts/bb-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ TESTS=(
crypto_schnorr_tests
crypto_sha256_tests
ecc_tests
join_split_example_proofs_inner_proof_data_tests
join_split_example_proofs_notes_tests
numeric_tests
plonk_tests
Expand Down
2 changes: 1 addition & 1 deletion cpp/src/barretenberg/crypto/schnorr/schnorr.tcc
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ schnorr_signature schnorr_construct_signature(const std::string& message, const
// method is overloaded to utilise a suitable entropy source
// (see https://github.com/WebAssembly/WASI/blob/main/phases/snapshot/docs.md)
//
// TODO: securely erase `k`
// TODO(https://github.com/AztecProtocol/barretenberg/issues/895): securely erase `k`
Fr k = Fr::random_element();

typename G1::affine_element R(G1::one * k);
Expand Down

This file was deleted.

This file was deleted.

Loading

0 comments on commit 82e2797

Please sign in to comment.