-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Agile Coretime Wiki Docs #5642
Agile Coretime Wiki Docs #5642
Conversation
Starter document
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.
Hey @DrW3RK I added some comment that could improve this page. It would be nice to start with an example and carry on that example through the sections.
docs/learn/learn-agile-coretime.md
Outdated
|
||
A timeslice represents the granularity the Coretime Chain schedules cores on the Relay Chain. | ||
The Coretime Chain announces a schedule for each core, comprising the next 80 relay | ||
chain blocks in advance. The region length of 28 days corresponds to 5,040 timeslices. |
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.
why is the 28 days important? Is this connected to bulk coretime?
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.
Yeah. Addressed it in the last commit, by rephrasing the sentence better.
docs/learn/learn-agile-coretime.md
Outdated
|
||
## Agile Coretime Implementation | ||
|
||
Coretime is managed through the [broker pallet](https://github.com/paritytech/polkadot-sdk/blob/eefd5fe4499515da66d088505093f75b5aa22550/substrate/frame/broker/src/lib.rs) |
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.
Interesting, so the coretime chain will support other things than just coretime brokerage? Is that why they did not call it coretime broker chain?
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.
Of course. Other functionalities are planned, like adding support for coretime credits, etc., which could be part of a broker pallet or some other pallet.
"why they did not call it coretime broker chain?"
This is partly answered in the "Product" section of FAQ uploaded. Also it is a lengthy name to have and some sort of branding decision for the system chain.
docs/learn/learn-agile-coretime.md
Outdated
|
||
- **Splitting**: the action of dividing a region into several regions with different start and end | ||
timeslices. | ||
- **Interlacing:** the action of dividing a region into the execution of different tasks for each |
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.
Can we give a practical real example here about splitting and interlacing? It is a bit hard to follow
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.
Yeah, this will be part of another PR that provides more info through guides and tutorials.
Thanks for your suggestions. I will incorporate them in the docs and then open it up for review. Right now, it is not ready for a review. |
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.
lgtm, I suggested conditional rendering for Kusama, and some minor edits
docs/learn/learn-agile-coretime.md
Outdated
slug: ../learn-agile-coretime | ||
--- | ||
|
||
Agile Coretime enables efficient utilization of Polkadot network resources and provides |
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.
Should we implement conditional rendering for Kusama? I guess this page will be added to Kusama Guide as well?
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.
Good Idea. But we shall do a separate PR for this.
docs/learn/learn-agile-coretime.md
Outdated
and envisioned in its [whitepaper](https://polkadot.network/whitepaper/). | ||
|
||
In Polkadot 1.0, the only way for a parachain to be secured by Polkadot was to rent a lease | ||
through an auction, which guaranteed parachain block validation for up-to two years. This involved |
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.
we could link the auction page here. Although that page will be in the archive, but I think it would be informative to the reader
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.
Added
Co-authored-by: Filippo <[email protected]>
Co-authored-by: Filippo <[email protected]>
Starter document for Agile Coretime and its terminology