8000 Framework for change proofs by rkuris · Pull Request #839 · ava-labs/firewood · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Framework for change proofs #839

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

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Framework for change proofs #839

wants to merge 10 commits into from

Conversation

rkuris
Copy link
Collaborator
@rkuris rkuris commented Apr 12, 2025

Adds some additional methods for iterating the trie. Still needed:

  • Some way for the iterator to skip identical branches
  • Complete the traversal algorithm

@rkuris rkuris force-pushed the rkuris/change-proofs branch from 2bd85d1 to c2035b5 Compare April 16, 2025 17:51
rkuris added 6 commits April 16, 2025 14:09
Adds some additional methods for iterating the trie. Still needed:
 - Some way for the iterator to skip identical branches
 - Complete the traversal algorithm
Added a new next_internal method that contains the shared iterationi
logic.
Makes error handling more consistent between Stream and Iterator.
While debugging, I couldn't figure out where the error was coming from,
so I revamped the error handling to include the filename and line number
everywhere an IO error could happen.
@rkuris rkuris force-pushed the rkuris/change-proofs branch from c2035b5 to 7a1242e Compare April 16, 2025 21:11
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
0