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

Agile Coretime Wiki Docs #5642

Merged
merged 13 commits into from
Mar 25, 2024
Merged

Agile Coretime Wiki Docs #5642

merged 13 commits into from
Mar 25, 2024

Conversation

DrW3RK
Copy link
Member

@DrW3RK DrW3RK commented Feb 28, 2024

Starter document for Agile Coretime and its terminology

@DrW3RK DrW3RK self-assigned this Feb 29, 2024
@DrW3RK DrW3RK added the A1 - In Progress Not ready for review yet. label Feb 29, 2024
Copy link
Contributor

@filippoweb3 filippoweb3 left a 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.

polkadot-wiki/sidebars.js Show resolved Hide resolved

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.
Copy link
Contributor

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?

Copy link
Member Author

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.


## Agile Coretime Implementation

Coretime is managed through the [broker pallet](https://github.com/paritytech/polkadot-sdk/blob/eefd5fe4499515da66d088505093f75b5aa22550/substrate/frame/broker/src/lib.rs)
Copy link
Contributor

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?

Copy link
Member Author

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.


- **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
Copy link
Contributor

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

Copy link
Member Author

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.

@DrW3RK
Copy link
Member Author

DrW3RK commented Mar 22, 2024

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.

@DrW3RK DrW3RK added A2 - Please Review Pull request is ready for review. and removed A1 - In Progress Not ready for review yet. labels Mar 25, 2024
Copy link
Contributor

@filippoweb3 filippoweb3 left a 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

slug: ../learn-agile-coretime
---

Agile Coretime enables efficient utilization of Polkadot network resources and provides
Copy link
Contributor

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?

Copy link
Member Author

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.

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
Copy link
Contributor

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Added

docs/learn/learn-agile-coretime.md Outdated Show resolved Hide resolved
docs/learn/learn-agile-coretime.md Outdated Show resolved Hide resolved
@DrW3RK DrW3RK merged commit d6fd417 into master Mar 25, 2024
2 checks passed
@DrW3RK DrW3RK deleted the agile-coretime-wiki-docs branch March 25, 2024 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A2 - Please Review Pull request is ready for review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants