Skip to content

chore: utils folder#16

Merged
twoeths merged 2 commits intomainfrom
te/refactor_utils_folder
Jun 18, 2025
Merged

chore: utils folder#16
twoeths merged 2 commits intomainfrom
te/refactor_utils_folder

Conversation

@twoeths
Copy link
Copy Markdown
Collaborator

@twoeths twoeths commented Jun 18, 2025

Motivation

  • All committee_indices, pubkey_index_map and shuffle are utils and we need to move all of them to utils folder to make the code structure cleaner

@github-actions
Copy link
Copy Markdown

Performance Report

✔️ no performance regression detected

Full benchmark results
Benchmark suite Current: 7047168 Previous: null Ratio
JS - unshuffleList - 16384 indices 916.24 us/op
Zig - unshuffleList - 16384 indices 601.11 us/op
JS - unshuffleList - 250000 indices 13.391 ms/op
Zig - unshuffleList - 250000 indices 9.0056 ms/op
JS - unshuffleList - 1000000 indices 52.248 ms/op
Zig - unshuffleList - 1000000 indices 35.546 ms/op
JS - computeSyncCommitteeIndices - 16384 indices 333.61 ms/op
Zig - computeSyncCommitteeIndices - 16384 indices 3.9987 ms/op
JS - computeSyncCommitteeIndices - 250000 indices 312.36 ms/op
Zig - computeSyncCommitteeIndices - 250000 indices 16.933 ms/op
JS - computeSyncCommitteeIndices - 1000000 indices 322.85 ms/op
Zig - computeSyncCommitteeIndices - 1000000 indices 31.295 ms/op
get values - 1000 294.00 ns/op
get values - naive - 1000 504.00 ns/op
set values - 1000 365.00 ns/op
set values - naive - 1000 601.00 ns/op
get values - 1000000 836.00 ns/op
get values - naive - 1000000 1.4610 us/op
set values - 1000000 848.00 ns/op
set values - naive - 1000000 2.0310 us/op

by benchmarkbot/action

@twoeths twoeths marked this pull request as ready for review June 18, 2025 01:49
@twoeths twoeths merged commit 76cc372 into main Jun 18, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant