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

Add a func to determine if Data is the minimum square #1517

Open
evan-forbes opened this issue Mar 22, 2023 · 4 comments
Open

Add a func to determine if Data is the minimum square #1517

evan-forbes opened this issue Mar 22, 2023 · 4 comments
Labels
good first issue Good for newcomers

Comments

@evan-forbes
Copy link
Member

celestia-node checks if the data will result in an empty square, which has a deterministic data root. We should add a function that determines that here so that they don't have to think about that logic.

@evan-forbes evan-forbes added the good first issue Good for newcomers label Mar 22, 2023
@evan-forbes evan-forbes added this to the Mainnet milestone Mar 22, 2023
@rootulp
Copy link
Collaborator

rootulp commented Apr 14, 2023

Is this effectively moving https://github.com/celestiaorg/celestia-node/blob/94cd2a6664a24336629caf12b7560cc7cb353f6e/share/empty.go#L62-L65 to an exported function in celestia-app?

@evan-forbes
Copy link
Member Author

this should just be a simple method that return true or false depending on if the data square is empty, so we're not creating a new square like the function above

@rootulp
Copy link
Collaborator

rootulp commented Apr 17, 2023

@evan-forbes
Copy link
Member Author

ahh I see, then I guess the easiest is to do what suggested by only implementing emptyDAH. Perhaps we could do both. Anything we can do to keep logic in the app that we are responsible for maintaining and not celestia-node

@evan-forbes evan-forbes removed this from the Mainnet milestone May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants