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.

Sign up for GitHub

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

Jump to bottom

PoT checkpoints in justifications #1948

Closed
nazar-pc opened this issue Sep 12, 2023 · 0 comments · Fixed by #1976
Closed

PoT checkpoints in justifications #1948

nazar-pc opened this issue Sep 12, 2023 · 0 comments · Fixed by #1976
Assignees
@nazar-pc
Labels
breaking-node This PR introduces breaking changes to the node implementation core Related to core protocol, may affect fundamentals improvement it is already working, but can be better node Node (service library/node app) performance Related to performance measurement or improvement
Milestone
Proof of Time

Comments

@nazar-pc
Copy link
Member

This will allow for more efficient PoT verification of blocks that contain otherwise unknown checkpoints/PoT fork.

Requires paritytech/polkadot-sdk#1211 to not panic.

This is technially an optimization, but a very important one.
@nazar-pc nazar-pc added improvement it is already working, but can be better node Node (service library/node app) core Related to core protocol, may affect fundamentals performance Related to performance measurement or improvement labels Sep 12, 2023
@nazar-pc nazar-pc added this to the Proof of Time milestone Sep 12, 2023
@nazar-pc nazar-pc self-assigned this Sep 12, 2023
@nazar-pc nazar-pc added the breaking-node This PR introduces breaking changes to the node implementation label Sep 12, 2023
@nazar-pc nazar-pc moved this from Todo to In Progress in Subspace core (node, farmer, etc.) Sep 17, 2023
@nazar-pc nazar-pc mentioned this issue Sep 18, 2023
Merged
1 task
@nazar-pc nazar-pc moved this from In Progress to Under Review in Subspace core (node, farmer, etc.) Sep 18, 2023
@github-project-automation github-project-automation bot moved this from Under Review to Closed in Subspace core (node, farmer, etc.) Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nazar-pc nazar-pc

Labels
breaking-node This PR introduces breaking changes to the node implementation core Related to core protocol, may affect fundamentals improvement it is already working, but can be better node Node (service library/node app) performance Related to performance measurement or improvement
Projects
Subspace core (node, farmer, etc.)
Status: Closed
Milestone
Proof of Time
Development

Successfully merging a pull request may close this issue.

PoT checkpoints in justifications autonomys/subspace
1 participant
@nazar-pc