-
Notifications
You must be signed in to change notification settings - Fork 285
[WIP] caps/eth.md: cell exchange #263
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
healthykim
wants to merge
33
commits into
ethereum:master
Choose a base branch
from
healthykim:bs/eth-70/ver-2
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 1 commit
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
e0bfa16
add: add eth70 spec
healthykim 5ba0075
fix: omit proof messaging
healthykim 556cc66
add: add a link to peerDAS
healthykim 4f29887
fix: restore Transactions msg spec
healthykim 75753f0
fix: rename ids to index
healthykim e558fa1
fix: specify the length of cells
healthykim ce56fc7
fix: integrate bitmap to hash announcement
healthykim 1d36625
fix: change cells message
healthykim efe662b
chore: fix typo
healthykim 9a8ff92
add: add explanation about bitmap granularity
healthykim 0896be2
add: add constraint in Cells message
healthykim 6caf1bc
add: modify intro
healthykim bb37e52
add: add behaviour
healthykim 6021b98
add detailed explanation of the Cells message
healthykim a902ea9
clarify meaning of cells field
healthykim c2a1f03
clarify meaning of cells field
healthykim 744324f
fix typo
healthykim eab5207
fix typo
healthykim 9e049e7
polish intro
healthykim bda330c
add more information about custody set
healthykim cafd6d3
edit intro
healthykim 24be46e
modify intro
healthykim 7d162ef
clarify cell index vs. tx better
cskiraly 390d9bc
fix typo
cskiraly 5e476bb
improve explanation
cskiraly 4161bd1
edit the number of cell indices set under pull behavior
healthykim 46fffe3
clarify scope of cells field
healthykim c064077
fix the incorrect restriction on the number of cells that can be requ…
healthykim 4cc7746
add parameter table and references to parameters
healthykim 6d926b1
clarify load balancing requirements
healthykim 2b12215
clarify the difference between p and MIN_P
healthykim bd2207a
fix typo
healthykim edb32f5
move peer disconnection to getcells from cells
healthykim File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next
Next commit
add: add eth70 spec
- Loading branch information
commit e0bfa16f45014aaa51259bf794242b5a15306f1e
There are no files selected for viewing
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are already eth/70 proposals in progress, so the number will be different
We can update at the end in the doc, but maybe remove from title to avoid confusion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed all eth/70 numbering in this PR and on doc so that we can add it back at the end.