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

feat: sp1 testing suite #1563

Merged
merged 56 commits into from
Oct 7, 2024
Merged

feat: sp1 testing suite #1563

merged 56 commits into from
Oct 7, 2024

Conversation

jtguibas
Copy link
Contributor

No description provided.

@jtguibas jtguibas marked this pull request as ready for review September 30, 2024 05:10
@jtguibas jtguibas marked this pull request as draft September 30, 2024 05:10
Copy link

github-actions bot commented Sep 30, 2024

SP1 Performance Test Results

Branch: john/testing-sweep
Commit: 99e9e02b
Author: jtguibas

program cycles execute (mHz) core (kHZ) compress (KHz) time success
fibonacci 11291 0.18 3.79 0.42 27s
ssz-withdrawals 2757356 11.62 99.34 35.72 1m17s
tendermint 12593597 5.57 210.23 102.38 2m5s

@jtguibas jtguibas changed the title some changes feat: sp1 testing suite Sep 30, 2024
@jtguibas jtguibas marked this pull request as ready for review September 30, 2024 05:33
jobs:
test-x86:
name: ${{ matrix.s3_folder }} (${{ matrix.mode }})
runs-on:
Copy link
Contributor

Choose a reason for hiding this comment

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

To handle the GPU runner, something like this:

runs-on: ${{ matrix.mode == 'cuda' && 'nvidia-gpu-linux-x64' || fromJSON('["runs-on", "runner=64cpu-linux-x64", "spot=false", "run-id=${{ github.run_id }}"]') }}

@jtguibas jtguibas changed the base branch from dev to tamir/v1.3.0-rc2 September 30, 2024 20:29
@jtguibas jtguibas merged commit 171be81 into tamir/v1.3.0-rc2 Oct 7, 2024
16 of 19 checks passed
@jtguibas jtguibas deleted the john/testing-sweep branch October 7, 2024 17:51
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.

4 participants