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

Feature/8407 spec ouroboros samasika consensus - skeleton and min window density #9307

Merged
merged 5 commits into from
Mar 29, 2022

Conversation

jspada
Copy link
Contributor

@jspada jspada commented Jul 28, 2021

This PR is for tracking the chainsafe spec's work for creating the intial consensus spec (breadth first) and deep-dive into window density specification.

Closes #8407

@jspada jspada force-pushed the feature/8407-spec-ouroboros-samasika-consensus branch from 08f3033 to 2307f60 Compare August 27, 2021 12:12
@jspada jspada changed the title Feature/8407 spec ouroboros samasika consensus Feature/8407 spec ouroboros samasika consensus - skeleton and min window density Oct 20, 2021
Copy link
Member

@nholland94 nholland94 left a comment

Choose a reason for hiding this comment

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

Mostly just have some small nits, but there were a couple of details that seem to be missing.

docs/specs/consensus/README.md Show resolved Hide resolved
docs/specs/consensus/README.md Outdated Show resolved Hide resolved
docs/specs/consensus/README.md Outdated Show resolved Hide resolved
docs/specs/consensus/README.md Outdated Show resolved Hide resolved
docs/specs/consensus/README.md Outdated Show resolved Hide resolved
docs/specs/consensus/README.md Outdated Show resolved Hide resolved
docs/specs/consensus/README.md Outdated Show resolved Hide resolved
docs/specs/consensus/README.md Show resolved Hide resolved
docs/specs/consensus/README.md Outdated Show resolved Hide resolved
@jspada jspada requested a review from vanishreerao February 28, 2022 18:10
Copy link
Contributor

@vanishreerao vanishreerao left a comment

Choose a reason for hiding this comment

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

looks good

@jspada jspada force-pushed the feature/8407-spec-ouroboros-samasika-consensus branch from 36e008e to a2ab609 Compare March 24, 2022 08:18
    * Add details about k-th predecessor edge case
    * Integrate review feedback from Matthew and Nathan
@jspada jspada added the ci-build-me Add this label to trigger a circle+buildkite build for this branch label Mar 29, 2022
@jspada jspada merged commit 008106a into compatible Mar 29, 2022
@jspada jspada deleted the feature/8407-spec-ouroboros-samasika-consensus branch March 29, 2022 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[mina spec] - Ouroboros Samasika consensus - breadth-first skeleton and sliding window density details
5 participants