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

specs: namespace padding shares #1344

Closed
rootulp opened this issue Feb 6, 2023 · 1 comment
Closed

specs: namespace padding shares #1344

rootulp opened this issue Feb 6, 2023 · 1 comment
Assignees
@rootulp
Labels
specs directly relevant to the specs

Comments

@rootulp
Copy link
Collaborator

rootulp commented Feb 6, 2023 •
edited
Loading

Context

If #1341 is merged, we need to update the specs-staging branch to state that a namespace padding share has a sequence start indicator of 1 and a sequence length of 0.
@rootulp rootulp added the specs directly relevant to the specs label Feb 6, 2023
@rootulp rootulp self-assigned this Feb 6, 2023
@rootulp rootulp mentioned this issue Feb 6, 2023
Merged
rootulp added a commit that referenced this issue Feb 7, 2023
@rootulp @cmwaters
feat!: differentiate namespaced padded shares (#1341)
a113f5a 
Implements the proposal in
#783 (comment).

Closes #1136
Closes #783
Opens #1344

---------

Co-authored-by: Callum Waters <[email protected]>
@rootulp rootulp changed the title specs: namespaced padded shares specs: namespace padding shares Feb 8, 2023
@rootulp rootulp mentioned this issue Feb 8, 2023
Merged
rootulp added a commit that referenced this issue Feb 8, 2023
@rootulp
docs: revise specs for padding shares (#1359)
0dc7d17 
Context: #1353
Closes #1344

## Screenshot

<img width="746" alt="Screenshot 2023-02-08 at 11 55 07 AM"
src="https://user-images.githubusercontent.com/3699047/217598202-15737680-d81b-4771-9940-75aa1830eaa2.png">
@rootulp
Copy link
Collaborator Author

rootulp commented Feb 9, 2023

Closed by #1359

@rootulp rootulp closed this as completed Feb 9, 2023
evan-forbes pushed a commit that referenced this issue Feb 27, 2023
@rootulp @cmwaters @evan-forbes
feat!: differentiate namespaced padded shares (#1341)
a4c7af0 
Implements the proposal in
#783 (comment).

Closes #1136
Closes #783
Opens #1344

---------

Co-authored-by: Callum Waters <[email protected]>
cmwaters added a commit to celestiaorg/go-square that referenced this issue Dec 14, 2023
@rootulp @cmwaters
feat!: differentiate namespaced padded shares (#1341)
01de228 
Implements the proposal in
celestiaorg/celestia-app#783 (comment).

Closes celestiaorg/celestia-app#1136
Closes celestiaorg/celestia-app#783
Opens celestiaorg/celestia-app#1344

---------

Co-authored-by: Callum Waters <[email protected]>
0xchainlover pushed a commit to celestia-org/celestia-app that referenced this issue Aug 1, 2024
@rootulp @cmwaters
feat!: differentiate namespaced padded shares (#1341)
7c64894 
Implements the proposal in
celestiaorg/celestia-app#783 (comment).

Closes celestiaorg/celestia-app#1136
Closes celestiaorg/celestia-app#783
Opens celestiaorg/celestia-app#1344

---------

Co-authored-by: Callum Waters <[email protected]>
0xchainlover pushed a commit to celestia-org/celestia-app that referenced this issue Aug 1, 2024
@rootulp
docs: revise specs for padding shares (#1359)
e99f5b6 
Context: celestiaorg/celestia-app#1353
Closes celestiaorg/celestia-app#1344

## Screenshot

<img width="746" alt="Screenshot 2023-02-08 at 11 55 07 AM"
src="https://user-images.githubusercontent.com/3699047/217598202-15737680-d81b-4771-9940-75aa1830eaa2.png">
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rootulp rootulp

Labels
specs directly relevant to the specs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rootulp