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

Add guest cycle count test with input from file. #1761

Open
wants to merge 10 commits into
base: nightly
Choose a base branch
from

Conversation

yaziciahmet
Copy link
Contributor

Description

Adds a test to both generate an input file for the guest circuit, and also the test to build and run the circuit with the input file. Input files are not committed to repo since it is above 500MB. If anyone wants or needs to test cycles, you can download it from our private S3.

Linked Issues

Testing

Describe how these changes were tested. If you've added new features, have you added unit tests?

Docs

Describe where this code is documented. If it changes a documented interface, have the docs been updated?

Copy link

codecov bot commented Jan 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.4%. Comparing base (347d641) to head (68b8b84).

Additional details and impacted files
Files with missing lines Coverage Δ
crates/risc0/src/host.rs 48.0% <ø> (ø)

... and 3 files with indirect coverage changes

@jfldde
Copy link
Contributor

jfldde commented Jan 31, 2025

Might want to look into the perf.yml and cycle-diff.sh script. This can either be used in addition to this PR or removed if not relevant anymore

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.

Add a new test to output the RISC0 cycle count
2 participants