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

Fix/repack in place 20241110 #641

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

JamesPiechota
Copy link
Collaborator

No description provided.

JamesPiechota and others added 8 commits November 6, 2024 20:31
when the `verify` launch option is set:
1. the node will launch from local state without joining the network
2. it will then iterate through all storage_modules validating different
   indices and proofs
3. any chunk that fails validation will be flagged as invalid
4. the next time the node is launchd with syncing enabled it will try to
   sync and pack the flagged chunks

This change also refactors the code for validating proofs. It introduces
a #chunk_proof record which could eventually replace the tuples and
maps that are currently used when dealing with proofs. This is just
an initial step, though, and the chunk_proof record is only used
internally (i.e. no updates to any serialized data structures)
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