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

feat: submit windowed posts repeatedly #624

Merged
merged 7 commits into from
Dec 10, 2024
Merged

feat: submit windowed posts repeatedly #624

merged 7 commits into from
Dec 10, 2024

Conversation

th7nder
Copy link
Contributor

@th7nder th7nder commented Dec 5, 2024

Description

It's a simple, at least for now, until it works. If it stops working, we can extend it.
Algorithm:

  1. When the pipeline starts, schedule a Windowed PoSt for each deadline (send a message to the queue).
  2. When the message is polled, it get the next not elapsed deadline info for an index from the chain.
  3. Waits till a PoSt can be sutmittted for the deadline.
  4. When the proof has been submitted, schedule another PoSt message to the pipeline. Start from 2. all over again.

just testnet && examples/rpc_publish.sh examples/test-data-big.txt

It keeps submitting a proof and works.

Closes #621.

Checklist

  • Make sure that you described what this change does.
  • Have you tested this solution?
  • Were there any alternative implementations considered?
  • Did you document new (or modified) APIs?
  • Follow-ups:

@th7nder th7nder added the enhancement New feature or request label Dec 5, 2024
@th7nder th7nder added this to the Phase 3 milestone Dec 5, 2024
@th7nder th7nder self-assigned this Dec 5, 2024
@th7nder th7nder marked this pull request as ready for review December 6, 2024 12:08
@th7nder th7nder requested a review from a team December 6, 2024 12:09
This was referenced Dec 6, 2024
@th7nder th7nder changed the title feat: submit windowed posts repeatadly feat: submit windowed posts repeatedly Dec 6, 2024
jmg-duarte
jmg-duarte previously approved these changes Dec 6, 2024
Copy link
Collaborator

@jmg-duarte jmg-duarte left a comment

Choose a reason for hiding this comment

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

Docs are kinda hard to understand in some places but the code looks sound.

I'll leave it up to you / someone else if they agree with my comments

pallets/storage-provider/src/deadline.rs Outdated Show resolved Hide resolved
pallets/storage-provider/src/deadline.rs Outdated Show resolved Hide resolved
aidan46
aidan46 previously approved these changes Dec 10, 2024
Copy link
Contributor

@aidan46 aidan46 left a comment

Choose a reason for hiding this comment

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

Nice work!

@th7nder th7nder added the ready for review Review is needed label Dec 10, 2024
@th7nder th7nder enabled auto-merge (squash) December 10, 2024 09:01
@th7nder th7nder added ready for review Review is needed and removed ready for review Review is needed labels Dec 10, 2024
@th7nder th7nder merged commit 7af27a4 into develop Dec 10, 2024
5 checks passed
@th7nder th7nder deleted the feat/621/schedule branch December 10, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ready for review Review is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Schedule Windowed PoSts repeatedly
4 participants