Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: Join-split example Part 1 #4965

Merged
merged 29 commits into from
Mar 8, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
5fdb0b4
Remove unused function
codygunton Mar 4, 2024
a2081d9
Delete more unused functions
codygunton Mar 4, 2024
18edf3d
Tweak tests to make them work without proof construction
codygunton Mar 5, 2024
472967c
Remove more proof construction functions
codygunton Mar 5, 2024
f37afd5
More unused stuff
codygunton Mar 5, 2024
e5de97c
No vk
codygunton Mar 5, 2024
9c4fd5d
New circuit hashing.
codygunton Mar 5, 2024
26201cb
Refactor value_note test
codygunton Mar 5, 2024
f01e70d
Remove unused file
codygunton Mar 5, 2024
2526f36
Create issue for security todo
codygunton Mar 5, 2024
226baf3
C++ syntax highlighting for .tcc
codygunton Mar 6, 2024
a7672ab
Use RelWithAssert
codygunton Mar 6, 2024
cc40461
New circuit hasing strategy
codygunton Mar 6, 2024
616dd96
Rename to reasonable names
codygunton Mar 6, 2024
72c7b26
Remove unused setup
codygunton Mar 6, 2024
d83be4b
Test rename
codygunton Mar 6, 2024
1bba00e
Move new code to CircuitChecker
codygunton Mar 6, 2024
5a9c66e
Revert "Use RelWithAssert"
codygunton Mar 6, 2024
73191f8
Remove ref to deleted binary
codygunton Mar 6, 2024
e541682
Rename fingerprint
codygunton Mar 6, 2024
12ce348
Revert alias change
codygunton Mar 6, 2024
8ae6a24
Does CI catch the bad hash?
codygunton Mar 6, 2024
0fc045c
Remove vk reference
codygunton Mar 6, 2024
29cb863
Revert "Does CI catch the bad hash?"
codygunton Mar 6, 2024
30253c9
Consts
codygunton Mar 6, 2024
398eb20
Cleanup dependencies
codygunton Mar 6, 2024
ed8af4d
Remove unused header
codygunton Mar 6, 2024
7a5b9cc
No relative pathing
codygunton Mar 6, 2024
ca178da
Rename variable for clarity
codygunton Mar 7, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions barretenberg/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": {
Copy link
Contributor

Choose a reason for hiding this comment

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

thats nice

"*.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 barretenberg/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
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just a serialization tests, not worth illustrating (tests deleted too).

Copy link
Contributor

Choose a reason for hiding this comment

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

we have proper serialization tests elsewhere that tests what this test used to test?

join_split_example_proofs_notes_tests
numeric_tests
plonk_tests
Expand Down
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
Loading