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

refactor(DA): syncing sampling types with spec #1128

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

andrussal
Copy link
Contributor

@andrussal andrussal commented Mar 11, 2025

1. What does this PR implement?

Naming in code and in specification aren’t 100% in sync. This PR should help to bridge this gap.


I am not sure how deep this should go. For example, in API we have the change: column_idx=>share_idx. Should this be done everywhere or only places that are visible to API callers?

2. Does the code have enough context to be clearly understood?

Yes

3. Who are the specification authors and who is accountable for this PR?

@andrussal

4. Is the specification accurate and complete?

N/A

5. Does the implementation introduce changes in the specification?

N/A

Checklist

  • 1. Description added.
  • 2. Context and links to Specification document(s) added.
  • 3. Main contact(s) (developers and specification authors) added
  • 4. Implementation and Specification are 100% in sync including changes. This is critical.
  • 5. Link PR to a specific milestone.

@andrussal andrussal added this to the Iteration 10 milestone Mar 11, 2025
@andrussal andrussal changed the title WIP: Syncing sampling types with spec WIP: (DA): syncing sampling types with spec Mar 11, 2025
@danielSanchezQ
Copy link
Collaborator

I am not sure how deep this should go. For example, in API we have the change: column_idx=>share_idx. Should this be done everywhere or only places that are visible to API callers?

Everywhere not just API. Also the struct names that are not on pair, for example:

DABlob => DAShare
DALightBlob => DALightShare
DASharedCommitments => DASharesCommitments

This ones are the one on top of my head at the moment.

@andrussal andrussal changed the title WIP: (DA): syncing sampling types with spec (DA): syncing sampling types with spec Mar 12, 2025
@andrussal andrussal force-pushed the da-sampling-naming-sync branch from 57992ce to 703b7e8 Compare March 12, 2025 19:46
@andrussal
Copy link
Contributor Author

I am not sure how deep this should go. For example, in API we have the change: column_idx=>share_idx. Should this be done everywhere or only places that are visible to API callers?

Everywhere not just API. Also the struct names that are not on pair, for example:

DABlob => DAShare
DALightBlob => DALightShare
DASharedCommitments => DASharesCommitments

This ones are the one on top of my head at the moment.

So I went all the way and number of files looks scary. But probably better get all done at once.

@danielSanchezQ danielSanchezQ changed the title (DA): syncing sampling types with spec refactor(DA): syncing sampling types with spec Mar 14, 2025
Copy link
Collaborator

@danielSanchezQ danielSanchezQ left a comment

Choose a reason for hiding this comment

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

It had to be done. Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants