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] Integrate sdk-rs and mock query args js cli #26

Merged
merged 6 commits into from
Jun 3, 2024

Conversation

ytham
Copy link
Contributor

@ytham ytham commented Jun 2, 2024

  • Adds sdk-rs mock-query-args CLI command
  • Adds AxiomStdRsTest to test rust circuit
  • Adds Average Age rust circuit in test/circuit-rs
  • Adds AccountAge.sol client

@ytham ytham requested a review from yi-sun June 2, 2024 01:55
@ytham
Copy link
Contributor Author

ytham commented Jun 2, 2024

tests are passing locally but unfortunately they are not passing in CI; investigating...

@ytham
Copy link
Contributor Author

ytham commented Jun 2, 2024

In CI:

    │   ├─ [0] VM::ffi(["cargo", "run", "--manifest-path", "test/circuit-rs/Cargo.toml", "--", "--input", "test/circuit-rs/data/account_age_input.json", "--data-path", "test/circuit-rs/data", "-k", "12", "-p", "***", "--to-stdout", "witness-gen"])
    │   │   └─ ← [Return] 0x

Local run:

    │   ├─ [0] VM::ffi(["cargo", "run", "--manifest-path", "test/circuit-rs/Cargo.toml", "--", "--input", "test/circuit-rs/data/account_age_input.json", "--data-path", "test/circuit-rs/data", "-k", "12", "-p", "***", "--to-stdout", "witness-gen"])
    │   │   └─ ← [Return] 0x5b0a202022307830303030303030303030303030303030303030303030303065663636336262306530623130393135373164616439373135393934626338316539663561326162222c0a202022307830303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030303030346538383233220a5d

Copy link
Contributor

@yi-sun yi-sun left a comment

Choose a reason for hiding this comment

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

LGTM!

@yi-sun yi-sun merged commit 8cf7d84 into feat/sdk-rs Jun 3, 2024
2 checks passed
@yi-sun yi-sun deleted the feat/sdk-rs-send branch June 3, 2024 07:08
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.

2 participants