Skip to content

feat: upgrade to zig 0.14 #8

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 6, 2025
Merged

feat: upgrade to zig 0.14 #8

merged 1 commit into from
May 6, 2025

Conversation

twoeths
Copy link
Collaborator

@twoeths twoeths commented May 6, 2025

twoeths
twoeths commented May 6, 2025
View reviewed changes
build.zig.zon
// It is redundant to include "zig" in this name because it is already
// within the Zig package namespace.
.name = "state-transition-z",
.name = .state_transition_z,
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this was generated by removing the above file and issue "zig init" again

@github-actions GitHub Actions
Copy link

github-actions bot commented May 6, 2025 •
edited
Loading

Performance Report

✔️ no performance regression detected

Full benchmark results
Benchmark suite Current: ef156b5 Previous: null Ratio
JS - unshuffleList - 16384 indices 2.0101 ms/op
Zig - unshuffleList - 16384 indices 714.36 us/op
JS - unshuffleList - 250000 indices 19.371 ms/op
Zig - unshuffleList - 250000 indices 27.605 ms/op
JS - unshuffleList - 1000000 indices 88.247 ms/op
Zig - unshuffleList - 1000000 indices 44.411 ms/op
get values - 1000 504.00 ns/op
get values - naive - 1000 1.0090 us/op
set values - 1000 486.00 ns/op
set values - naive - 1000 721.00 ns/op
get values - 1000000 1.4950 us/op
get values - naive - 1000000 2.1380 us/op
set values - 1000000 1.0910 us/op
set values - naive - 1000000 2.6860 us/op
JS - computeSyncCommitteeIndices - 16384 indices 520.99 ms/op
Zig - computeSyncCommitteeIndices - 16384 indices 4.9562 ms/op
JS - computeSyncCommitteeIndices - 250000 indices 501.00 ms/op
Zig - computeSyncCommitteeIndices - 250000 indices 18.888 ms/op
JS - computeSyncCommitteeIndices - 1000000 indices 521.46 ms/op
Zig - computeSyncCommitteeIndices - 1000000 indices 33.206 ms/op

by benchmarkbot/action

@twoeths twoeths marked this pull request as ready for review May 6, 2025 02:31
@twoeths twoeths merged commit 8dfd280 into main May 6, 2025
4 checks passed
@twoeths twoeths deleted the te/zig_0.14 branch May 6, 2025 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@twoeths