FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix: write final pos back by kunxian-xia · Pull Request #16 · scroll-tech/openvm · GitHub

Fix: write final pos back - #16

Merged
kunxian-xia merged 6 commits into
feat/v1.4.1-scroll-extfrom
fix/write_final_pos
Nov 26, 2025
Merged

Fix: write final pos back#16
kunxian-xia merged 6 commits into
feat/v1.4.1-scroll-extfrom
fix/write_final_pos

Conversation

kunxian-xia commented Nov 26, 2025
edited
Loading

Copy link
Copy Markdown

Summary

Issue: we forget to write final sponge state position back in #12.

  • We add a new function in DuplexChallengerVariable called observe_slice_opt which is equivalent to observe_slice but costs much less cycles.
  • Test it in unit test test_multi_observe.
    cmd: RUST_LOG=info,p3_=warn cargo test --release --test recursion test_multi_observe --package openvm-native-recursion --features "metrics,openvm-circuit/stark-debug,cuda" -- --nocapture.

kunxian-xia merged commit d73de3e into feat/v1.4.1-scroll-ext Nov 26, 2025
kunxian-xia deleted the fix/write_final_pos branch November 26, 2025 06:28
kunxian-xia mentioned this pull request Nov 26, 2025
8 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL