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

chore(rln): expose leaves_set api #213

Merged
merged 1 commit into from
Sep 12, 2023
Merged

chore(rln): expose leaves_set api #213

merged 1 commit into from
Sep 12, 2023

Conversation

rymnc
Copy link
Contributor

@rymnc rymnc commented Sep 12, 2023

Exposes the leaves_set api to allow consuming applications to have more metadata about the tree

@rymnc rymnc added the track:rln RLN Track - (Secure Messaging/Applied ZK), relay and applications label Sep 12, 2023
@rymnc rymnc self-assigned this Sep 12, 2023
@github-actions
Copy link

Benchmark for e91aa28

Click to view benchmark
Test Base PR %
FullMerkleTree::delete 1239.3±4.23ns 1245.0±0.40ns +0.46%
FullMerkleTree::get 4.0±0.00ns 4.0±0.00ns 0.00%
FullMerkleTree::override_range 3.1±0.00µs 3.2±0.00µs +3.23%
FullMerkleTree::set 1238.9±0.46ns 1245.8±2.27ns +0.56%
OptimalMerkleTree::compute_root 1390.9±6.36ns 1386.1±1.09ns -0.35%
OptimalMerkleTree::delete 1385.7±3.90ns 1381.6±0.78ns -0.30%
OptimalMerkleTree::get 25.9±0.01ns 25.6±0.02ns -1.16%
OptimalMerkleTree::override_range 7.1±0.01µs 7.2±0.05µs +1.41%
OptimalMerkleTree::set 1383.8±2.90ns 1379.5±0.97ns -0.31%

@github-actions
Copy link

Benchmark for e91aa28

Click to view benchmark
Test Base PR %
Pmtree::compute_root 2.0±0.01ns 2.0±0.02ns 0.00%
Pmtree::get 464.8±3.16ns 461.6±0.45ns -0.69%
Pmtree::override_range 180.9±10.57µs 176.2±8.65µs -2.60%
Pmtree::set 70.0±1.10µs 69.9±0.06µs -0.14%
Pmtree:delete 69.7±0.08µs 70.0±1.87µs +0.43%

@github-actions
Copy link

Benchmark for 8007266

Click to view benchmark
Test Base PR %
FullMerkleTree::delete 1423.2±57.59ns 1438.9±52.99ns +1.10%
FullMerkleTree::get 4.1±0.15ns 4.1±0.22ns 0.00%
FullMerkleTree::override_range 3.6±0.17µs 3.5±0.14µs -2.78%
FullMerkleTree::set 1406.5±43.02ns 1520.3±87.00ns +8.09%
OptimalMerkleTree::compute_root 1646.4±65.55ns 1709.1±63.54ns +3.81%
OptimalMerkleTree::delete 1602.4±71.39ns 1721.8±90.64ns +7.45%
OptimalMerkleTree::get 30.8±3.16ns 32.6±2.19ns +5.84%
OptimalMerkleTree::override_range 8.3±0.34µs 8.8±0.51µs +6.02%
OptimalMerkleTree::set 1594.7±72.25ns 1832.4±244.82ns +14.91%

@github-actions
Copy link

Benchmark for 8007266

Click to view benchmark
Test Base PR %
Pmtree::compute_root 2.0±0.02ns 2.0±0.02ns 0.00%
Pmtree::get 460.6±0.34ns 475.9±0.33ns +3.32%
Pmtree::override_range 184.1±11.23µs 184.3±15.91µs +0.11%
Pmtree::set 70.1±0.63µs 69.5±0.06µs -0.86%
Pmtree:delete 69.8±0.05µs 69.7±0.14µs -0.14%

@rymnc rymnc merged commit f8fc455 into master Sep 12, 2023
16 checks passed
@rymnc rymnc deleted the leaves-set-ffi-api branch September 12, 2023 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
track:rln RLN Track - (Secure Messaging/Applied ZK), relay and applications
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant