feat: Sorted execution trace #5252

ledwards2225 · 2024-03-15T15:33:44Z

This work results in an execution trace fully sorted by gate type.

Prior to this PR, all of the infrastructure was added to construct and process a sorted execution trace. Each builder has a blocks object which essentially holds a {wires, selectors} pair for each gate type. Up until now, all gates were being added into blocks.main, which is equivalent to what we've always done. This PR simply adds gates into their appropriate specialized block, e.g. arithmetic gates are added into blocks.arithmetic and auxiliary gates are added into blocks.aux. After being processed in the ExecutionTrace class, this results in an execution trace sorted by gate type.

Note: This PR adds dummy gates in several new locations to account for the fact that some gates of a particular type were previously reading into a subsequent gate of different type, which breaks once the gates are sorted by type.
Note: This PR does not include any logic for taking advantage of the sorted structure.

Closes AztecProtocol/barretenberg#867 (gates are sorted)
Closes AztecProtocol/barretenberg#873 (no more gate interleaving assumptions, except ones now identified in more specific issues)
Closes AztecProtocol/barretenberg#910 (gate type summary via blocks.summarize())

… blocks

github-actions · 2024-03-15T16:02:26Z

Changes to circuit sizes

Generated at commit: 15d071f4795d8c5fbd9d2e681ed4d97c658a9c28, compared to commit: 10ef9702c43d36afd334a78df26fe0301c2ac001

🧾 Summary (100% most significant diffs)

Program	%	Circuit size (+/-)	%
rollup_base	0.00%	-4 ✅	-0.00%
private_kernel_inner	0.00%	-4 ✅	-0.00%
private_kernel_tail	0.00%	-7 ✅	-0.00%
public_kernel_tail	0.00%	-3 ✅	-0.00%
public_kernel_app_logic	0.00%	-3 ✅	-0.00%
private_kernel_init	0.00%	-4 ✅	-0.00%
public_kernel_teardown	0.00%	-3 ✅	-0.00%
public_kernel_setup	0.00%	-3 ✅	-0.00%

Full diff report 👇

Program	ACIR opcodes (+/-)	%	Circuit size (+/-)	%
rollup_base	171,450 (0)	0.00%	1,730,656 (-4)	-0.00%
private_kernel_inner	95,996 (0)	0.00%	521,442 (-4)	-0.00%
private_kernel_tail	373,475 (0)	0.00%	839,580 (-7)	-0.00%
public_kernel_tail	124,967 (0)	0.00%	358,964 (-3)	-0.00%
public_kernel_app_logic	56,202 (0)	0.00%	316,867 (-3)	-0.00%
private_kernel_init	48,706 (0)	0.00%	348,989 (-4)	-0.00%
public_kernel_teardown	29,516 (0)	0.00%	184,407 (-3)	-0.00%
public_kernel_setup	29,408 (0)	0.00%	184,298 (-3)	-0.00%

AztecBot · 2024-03-18T15:55:33Z

Benchmark results

No metrics with a significant change found.

Detailed results

All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.

This benchmark source data is available in JSON format on S3 here.

Values are compared against data from master at commit 10ef9702 and shown if the difference exceeds 1%.

L2 block published to L1

Each column represents the number of txs on an L2 block published to L1.

Metric	8 txs	32 txs	64 txs
l1_rollup_calldata_size_in_bytes	5,668	18,820	36,356
l1_rollup_calldata_gas	66,292	239,080	469,112
l1_rollup_execution_gas	647,390	929,438	1,305,292
l2_block_processing_time_in_ms	1,271 (-2%)	4,569 (-4%)	9,368
note_successful_decrypting_time_in_ms	176 (+2%)	523 (-6%)	976 (-2%)
note_trial_decrypting_time_in_ms	81.8 (+9%)	42.2 (-46%)	109 (-13%)
l2_block_building_time_in_ms	18,110	68,932	137,191
l2_block_rollup_simulation_time_in_ms	8,109 (+1%)	28,680	56,796
l2_block_public_tx_process_time_in_ms	9,979 (-1%)	40,197 (+1%)	80,299

L2 chain processing

Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric	5 blocks	10 blocks
node_history_sync_time_in_ms	14,770 (+1%)	28,576 (+7%)
note_history_successful_decrypting_time_in_ms	1,290 (+5%)	2,582 (+4%)
note_history_trial_decrypting_time_in_ms	100 (+65%)	171 (-7%)
node_database_size_in_bytes	18,620,496	34,992,208
pxe_database_size_in_bytes	29,859	59,414

Circuits stats

Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit	circuit_simulation_time_in_ms	circuit_input_size_in_bytes	circuit_output_size_in_bytes
private-kernel-init	280	44,366	28,244
private-kernel-ordering	214	52,868	14,326
base-parity	1,928 (+8%)	128	311
base-rollup	718	165,787	925
root-parity	1,464 (-10%)	1,244	311
root-rollup	52.2 (+1%)	4,487	725
private-kernel-inner	643	73,771	28,244
public-kernel-app-logic	456 (+3%)	35,260	28,215
public-kernel-tail	176 (+2%)	40,926	28,215
merge-rollup	8.32	2,696	925

Tree insertion stats

The duration to insert a fixed batch of leaves into each tree type.

Metric	1 leaves	16 leaves	64 leaves	128 leaves	512 leaves	1024 leaves	2048 leaves	4096 leaves	32 leaves
batch_insert_into_append_only_tree_16_depth_ms	10.0	16.2 (+1%)	N/A	N/A	N/A	N/A	N/A	N/A	N/A
batch_insert_into_append_only_tree_16_depth_hash_count	16.8	31.6	N/A	N/A	N/A	N/A	N/A	N/A	N/A
batch_insert_into_append_only_tree_16_depth_hash_ms	0.585	0.500 (+1%)	N/A	N/A	N/A	N/A	N/A	N/A	N/A
batch_insert_into_append_only_tree_32_depth_ms	N/A	N/A	51.4 (+12%)	72.4	231	447	874 (-1%)	1,731	N/A
batch_insert_into_append_only_tree_32_depth_hash_count	N/A	N/A	96.0	159	543	1,055	2,079	4,127	N/A
batch_insert_into_append_only_tree_32_depth_hash_ms	N/A	N/A	0.528 (+12%)	0.447	0.422	0.419 (+1%)	0.415 (-1%)	0.415	N/A
batch_insert_into_indexed_tree_20_depth_ms	N/A	N/A	59.1 (+7%)	107 (-1%)	338 (-1%)	665 (+1%)	1,317 (-1%)	2,620	N/A
batch_insert_into_indexed_tree_20_depth_hash_count	N/A	N/A	104	207	691	1,363	2,707	5,395	N/A
batch_insert_into_indexed_tree_20_depth_hash_ms	N/A	N/A	0.529 (+8%)	0.483	0.460	0.458 (+1%)	0.457	0.454	N/A
batch_insert_into_indexed_tree_40_depth_ms	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	63.5 (+4%)
batch_insert_into_indexed_tree_40_depth_hash_count	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	109
batch_insert_into_indexed_tree_40_depth_hash_ms	N/A	N/A	N/A	N/A	N/A	N/A	N/A	N/A	0.556 (+4%)

Miscellaneous

Transaction sizes based on how many contract classes are registered in the tx.

Metric	0 registered classes
tx_size_in_bytes	22,012

Transaction processing duration by data writes.

Metric	0 new note hashes	1 new note hashes
tx_pxe_processing_time_ms	3,203 (-1%)	1,743

Metric	0 public data writes	1 public data writes
tx_sequencer_processing_time_ms	11.5	1,269 (+3%)

ledwards2225 · 2024-03-18T18:40:18Z

barretenberg/cpp/src/barretenberg/circuit_checker/circuit_checker.cpp

+    bool result = true;
+    // TODO(https://github.com/AztecProtocol/barretenberg/issues/870): Currently we check all relations for each block.
+    // Once sorting is complete, is will be sufficient to check only the relevant relation(s) per block.
+    size_t block_idx = 0;


Previously I was only checking the main block (which contained all gates). Now I'm looping over all blocks and checking all relations on each one.

ledwards2225 · 2024-03-18T18:41:29Z

barretenberg/cpp/src/barretenberg/goblin/mock_circuits.hpp

@@ -62,7 +62,7 @@ class GoblinMockCircuits {
    {
        // Determine number of times to execute the below operations that constitute the mock circuit logic. Note that
        // the circuit size does not scale linearly with number of iterations due to e.g. amortization of lookup costs
-        const size_t NUM_ITERATIONS_LARGE = 13; // results in circuit size 2^19 (521327 gates)
+        const size_t NUM_ITERATIONS_LARGE = 12; // results in circuit size 2^19 (502238 gates)


Had to reduce this due to dummy gates increasing sizes of some circuits

barretenberg/cpp/src/barretenberg/proof_system/arithmetization/arithmetization.hpp

ledwards2225 · 2024-03-18T18:44:48Z

barretenberg/cpp/src/barretenberg/proof_system/circuit_builder/ultra_circuit_builder.cpp

-    }
-    check_selector_length_consistency();
-    ++this->num_gates;
+    create_dummy_gate(block, in.x2, in.x3, in.y3, in.y2);


Here I'm just using the create_dummy_gate method instead of doing it manually

ledwards2225 · 2024-03-18T18:45:44Z

barretenberg/cpp/src/barretenberg/proof_system/circuit_builder/ultra_circuit_builder.cpp

-        blocks.main.pad_additional();
-    }
-    check_selector_length_consistency();
+    create_add_gate({ variable_index[0], this->zero_idx, this->zero_idx, 1, 0, 0, -start });


No change, just using create_add_gate instead of the manual thing

ledwards2225 · 2024-03-18T18:46:28Z

barretenberg/cpp/src/barretenberg/proof_system/circuit_builder/ultra_circuit_builder.cpp

-                          0,
-                          -end });
+    create_dummy_gate(block, variable_index[variable_index.size() - 1], this->zero_idx, this->zero_idx, this->zero_idx);
+    create_add_gate({ variable_index[variable_index.size() - 1], this->zero_idx, this->zero_idx, 1, 0, 0, -end });


No actual change here, just using create_add_gate instead since there was no need for the "big add gate" functionality

ledwards2225 · 2024-03-18T18:49:36Z

barretenberg/cpp/src/barretenberg/proof_system/circuit_builder/ultra_circuit_builder.hpp

@@ -313,7 +313,7 @@ class UltraCircuitBuilder_ : public CircuitBuilderBase<typename Arithmetization_
    UltraCircuitBuilder_(const size_t size_hint = 0)
        : CircuitBuilderBase<FF>(size_hint)
    {
-        blocks.main.reserve(size_hint);
+        // TODO(https://github.com/AztecProtocol/barretenberg/issues/870): reserve space in blocks here somehow?


Leaving this as a TODO. I previously looked at how important this reserve was and it turns out the answer is that it's entirely negligible.

barretenberg/cpp/src/barretenberg/circuit_checker/circuit_checker.cpp

barretenberg/cpp/src/barretenberg/proof_system/arithmetization/arithmetization.hpp

Rumata888

Chunky. Left very few comments. LGTM

🤖 I have created a release *beep* *boop* --- <details><summary>aztec-package: 0.30.1</summary> ## [0.30.1](aztec-package-v0.30.0...aztec-package-v0.30.1) (2024-03-20) ### Miscellaneous * **aztec-package:** Synchronize aztec-packages versions </details> <details><summary>barretenberg.js: 0.30.1</summary> ## [0.30.1](barretenberg.js-v0.30.0...barretenberg.js-v0.30.1) (2024-03-20) ### Miscellaneous * **barretenberg.js:** Synchronize aztec-packages versions </details> <details><summary>aztec-cli: 0.30.1</summary> ## [0.30.1](aztec-cli-v0.30.0...aztec-cli-v0.30.1) (2024-03-20) ### Miscellaneous * **aztec-cli:** Synchronize aztec-packages versions </details> <details><summary>aztec-packages: 0.30.1</summary> ## [0.30.1](aztec-packages-v0.30.0...aztec-packages-v0.30.1) (2024-03-20) ### Features * Add CMOV instruction to brillig and brillig gen ([#5308](#5308)) ([208abbb](208abbb)) * **avm:** Indirect memory support for arithmetic/bitwise opcodes ([#5328](#5328)) ([d5ffa17](d5ffa17)), closes [#5273](#5273) * **avm:** Indirect memory support for MOV ([#5257](#5257)) ([10ef970](10ef970)), closes [#5205](#5205) * Merge SMT Terms in one class ([#5254](#5254)) ([f5c9b0f](f5c9b0f)) * Sorted execution trace ([#5252](#5252)) ([a216759](a216759)) ### Bug Fixes * Fix recursion tests and reinstate in CI ([#5300](#5300)) ([96c6f21](96c6f21)) * Skip uniswap l1 tests ([#5334](#5334)) ([7a56941](7a56941)) * Update smt_verification README.md ([#5332](#5332)) ([46b15e3](46b15e3)) ### Miscellaneous * Avm team as generated codeowners ([#5325](#5325)) ([06d2786](06d2786)) * No Translator composer ([#5202](#5202)) ([c8897ca](c8897ca)) * Remove toy vm files ([#5326](#5326)) ([d940356](d940356)) * Replace relative paths to noir-protocol-circuits ([ea2ac09](ea2ac09)) </details> <details><summary>barretenberg: 0.30.1</summary> ## [0.30.1](barretenberg-v0.30.0...barretenberg-v0.30.1) (2024-03-20) ### Features * Add CMOV instruction to brillig and brillig gen ([#5308](#5308)) ([208abbb](208abbb)) * **avm:** Indirect memory support for arithmetic/bitwise opcodes ([#5328](#5328)) ([d5ffa17](d5ffa17)), closes [#5273](#5273) * **avm:** Indirect memory support for MOV ([#5257](#5257)) ([10ef970](10ef970)), closes [#5205](#5205) * Merge SMT Terms in one class ([#5254](#5254)) ([f5c9b0f](f5c9b0f)) * Sorted execution trace ([#5252](#5252)) ([a216759](a216759)) ### Bug Fixes * Fix recursion tests and reinstate in CI ([#5300](#5300)) ([96c6f21](96c6f21)) * Update smt_verification README.md ([#5332](#5332)) ([46b15e3](46b15e3)) ### Miscellaneous * No Translator composer ([#5202](#5202)) ([c8897ca](c8897ca)) * Remove toy vm files ([#5326](#5326)) ([d940356](d940356)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release *beep* *boop* --- <details><summary>aztec-package: 0.30.1</summary> ## [0.30.1](AztecProtocol/aztec-packages@aztec-package-v0.30.0...aztec-package-v0.30.1) (2024-03-20) ### Miscellaneous * **aztec-package:** Synchronize aztec-packages versions </details> <details><summary>barretenberg.js: 0.30.1</summary> ## [0.30.1](AztecProtocol/aztec-packages@barretenberg.js-v0.30.0...barretenberg.js-v0.30.1) (2024-03-20) ### Miscellaneous * **barretenberg.js:** Synchronize aztec-packages versions </details> <details><summary>aztec-cli: 0.30.1</summary> ## [0.30.1](AztecProtocol/aztec-packages@aztec-cli-v0.30.0...aztec-cli-v0.30.1) (2024-03-20) ### Miscellaneous * **aztec-cli:** Synchronize aztec-packages versions </details> <details><summary>aztec-packages: 0.30.1</summary> ## [0.30.1](AztecProtocol/aztec-packages@aztec-packages-v0.30.0...aztec-packages-v0.30.1) (2024-03-20) ### Features * Add CMOV instruction to brillig and brillig gen ([#5308](AztecProtocol/aztec-packages#5308)) ([208abbb](AztecProtocol/aztec-packages@208abbb)) * **avm:** Indirect memory support for arithmetic/bitwise opcodes ([#5328](AztecProtocol/aztec-packages#5328)) ([d5ffa17](AztecProtocol/aztec-packages@d5ffa17)), closes [#5273](AztecProtocol/aztec-packages#5273) * **avm:** Indirect memory support for MOV ([#5257](AztecProtocol/aztec-packages#5257)) ([10ef970](AztecProtocol/aztec-packages@10ef970)), closes [#5205](AztecProtocol/aztec-packages#5205) * Merge SMT Terms in one class ([#5254](AztecProtocol/aztec-packages#5254)) ([f5c9b0f](AztecProtocol/aztec-packages@f5c9b0f)) * Sorted execution trace ([#5252](AztecProtocol/aztec-packages#5252)) ([a216759](AztecProtocol/aztec-packages@a216759)) ### Bug Fixes * Fix recursion tests and reinstate in CI ([#5300](AztecProtocol/aztec-packages#5300)) ([96c6f21](AztecProtocol/aztec-packages@96c6f21)) * Skip uniswap l1 tests ([#5334](AztecProtocol/aztec-packages#5334)) ([7a56941](AztecProtocol/aztec-packages@7a56941)) * Update smt_verification README.md ([#5332](AztecProtocol/aztec-packages#5332)) ([46b15e3](AztecProtocol/aztec-packages@46b15e3)) ### Miscellaneous * Avm team as generated codeowners ([#5325](AztecProtocol/aztec-packages#5325)) ([06d2786](AztecProtocol/aztec-packages@06d2786)) * No Translator composer ([#5202](AztecProtocol/aztec-packages#5202)) ([c8897ca](AztecProtocol/aztec-packages@c8897ca)) * Remove toy vm files ([#5326](AztecProtocol/aztec-packages#5326)) ([d940356](AztecProtocol/aztec-packages@d940356)) * Replace relative paths to noir-protocol-circuits ([ea2ac09](AztecProtocol/aztec-packages@ea2ac09)) </details> <details><summary>barretenberg: 0.30.1</summary> ## [0.30.1](AztecProtocol/aztec-packages@barretenberg-v0.30.0...barretenberg-v0.30.1) (2024-03-20) ### Features * Add CMOV instruction to brillig and brillig gen ([#5308](AztecProtocol/aztec-packages#5308)) ([208abbb](AztecProtocol/aztec-packages@208abbb)) * **avm:** Indirect memory support for arithmetic/bitwise opcodes ([#5328](AztecProtocol/aztec-packages#5328)) ([d5ffa17](AztecProtocol/aztec-packages@d5ffa17)), closes [#5273](AztecProtocol/aztec-packages#5273) * **avm:** Indirect memory support for MOV ([#5257](AztecProtocol/aztec-packages#5257)) ([10ef970](AztecProtocol/aztec-packages@10ef970)), closes [#5205](AztecProtocol/aztec-packages#5205) * Merge SMT Terms in one class ([#5254](AztecProtocol/aztec-packages#5254)) ([f5c9b0f](AztecProtocol/aztec-packages@f5c9b0f)) * Sorted execution trace ([#5252](AztecProtocol/aztec-packages#5252)) ([a216759](AztecProtocol/aztec-packages@a216759)) ### Bug Fixes * Fix recursion tests and reinstate in CI ([#5300](AztecProtocol/aztec-packages#5300)) ([96c6f21](AztecProtocol/aztec-packages@96c6f21)) * Update smt_verification README.md ([#5332](AztecProtocol/aztec-packages#5332)) ([46b15e3](AztecProtocol/aztec-packages@46b15e3)) ### Miscellaneous * No Translator composer ([#5202](AztecProtocol/aztec-packages#5202)) ([c8897ca](AztecProtocol/aztec-packages@c8897ca)) * Remove toy vm files ([#5326](AztecProtocol/aztec-packages#5326)) ([d940356](AztecProtocol/aztec-packages@d940356)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

ledwards2225 added 30 commits March 6, 2024 17:59

initial PoC; addition gates in their own block; checker processes all…

dd01b70

… blocks

minor non changing updates

cd97781

update fibbo test

b0d944a

move some arith gates into their block

9f9db7b

update ecc double in the same way as ecc add

aac0c4e

roll back some of those

0216b1f

poly

8eeb03c

fix fidget test

79aa5db

fix witness and elliptic double

29ea42f

lookups

d3a8b80

sort (but not sort with edges)

dc4121d

use block syntax but no sorting change

c3e1128

Merge branch 'master' into lde/gate_sorting

f7af9d8

separate nonzero poly gates into individual gates/blocks

def3bdc

rephrase as dummy gate, no expected change

be60f07

dummy gates in nnf mul make tests pass w big add in arith

669d0ac

turn off dummy gates due to doubling in acir double verify proof circuit

df89d75

use add gate explicitly in sort with edges

7ee58d7

move sort with edges to sort block

a3401b6

big add is arithmetic

2eadd1a

Merge branch 'master' into lde/gate_sorting

7bf371a

move dummy gates to exactly where theyre needed

cbb027e

add a required dummy in bigfield

c05a8b0

update pinning and big mul to arithmetic

44991bb

use pop wires syntax, no change expected

d35dca4

move ecc add to elliptic

7d13b04

nnf add and subract are arithmetic

5af392f

move aux gates to aux

ba502f9

move poseidon int and ext to respective blocks

8a895c0

split poseidon in guh add gates

72f0e9b

ledwards2225 marked this pull request as ready for review March 15, 2024 15:35

reinstate double v prove for js with test prints

46cd7dd

ledwards2225 added 5 commits March 18, 2024 17:08

manual add gate added to aux block to avoid need for dummy

cc62695

cleanup and add TODO

374330a

cleanup TODOs

cb906a3

Merge branch 'master' into lde/sorted_trace

9da509f

remove old WORKTODO

b42dfff

ledwards2225 requested a review from Rumata888 March 18, 2024 18:38

ledwards2225 commented Mar 18, 2024

View reviewed changes

barretenberg/cpp/src/barretenberg/proof_system/arithmetization/arithmetization.hpp Outdated Show resolved Hide resolved

ledwards2225 commented Mar 18, 2024

View reviewed changes

ledwards2225 self-assigned this Mar 18, 2024

Rumata888 reviewed Mar 19, 2024

View reviewed changes

barretenberg/cpp/src/barretenberg/circuit_checker/circuit_checker.cpp Outdated Show resolved Hide resolved

Rumata888 reviewed Mar 19, 2024

View reviewed changes

barretenberg/cpp/src/barretenberg/proof_system/arithmetization/arithmetization.hpp Show resolved Hide resolved

Rumata888 approved these changes Mar 19, 2024

View reviewed changes

ledwards2225 added 3 commits March 19, 2024 21:28

address Keshas comments

6675126

Merge branch 'master' into lde/sorted_trace

6c4b037

name fix

55f1dec

ledwards2225 enabled auto-merge (squash) March 19, 2024 21:45

ledwards2225 merged commit a216759 into master Mar 19, 2024
93 of 95 checks passed

ledwards2225 deleted the lde/sorted_trace branch March 19, 2024 22:17

AztecBot mentioned this pull request Mar 19, 2024

chore(master): Release 0.30.1 #5322

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Sorted execution trace #5252

feat: Sorted execution trace #5252

ledwards2225 commented Mar 15, 2024 •

edited

Loading

github-actions bot commented Mar 15, 2024 •

edited

Loading

AztecBot commented Mar 18, 2024 •

edited

Loading

L2 block published to L1

L2 chain processing

Circuits stats

Tree insertion stats

Miscellaneous

ledwards2225 Mar 18, 2024

ledwards2225 Mar 18, 2024

ledwards2225 Mar 18, 2024 •

edited

Loading

ledwards2225 Mar 18, 2024

ledwards2225 Mar 18, 2024 •

edited

Loading

ledwards2225 Mar 18, 2024

Rumata888 left a comment

feat: Sorted execution trace #5252

feat: Sorted execution trace #5252

Conversation

ledwards2225 commented Mar 15, 2024 • edited Loading

github-actions bot commented Mar 15, 2024 • edited Loading

Changes to circuit sizes

🧾 Summary (100% most significant diffs)

AztecBot commented Mar 18, 2024 • edited Loading

Benchmark results

L2 block published to L1

L2 chain processing

Circuits stats

Tree insertion stats

Miscellaneous

ledwards2225 Mar 18, 2024

Choose a reason for hiding this comment

ledwards2225 Mar 18, 2024

Choose a reason for hiding this comment

ledwards2225 Mar 18, 2024 • edited Loading

Choose a reason for hiding this comment

ledwards2225 Mar 18, 2024

Choose a reason for hiding this comment

ledwards2225 Mar 18, 2024 • edited Loading

Choose a reason for hiding this comment

ledwards2225 Mar 18, 2024

Choose a reason for hiding this comment

Rumata888 left a comment

Choose a reason for hiding this comment

ledwards2225 commented Mar 15, 2024 •

edited

Loading

github-actions bot commented Mar 15, 2024 •

edited

Loading

AztecBot commented Mar 18, 2024 •

edited

Loading

ledwards2225 Mar 18, 2024 •

edited

Loading

ledwards2225 Mar 18, 2024 •

edited

Loading