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

Change endorsements #2976

Closed
qdrn opened this issue Sep 7, 2022 · 8 comments
Closed

Change endorsements #2976

qdrn opened this issue Sep 7, 2022 · 8 comments
Assignees

Comments

@qdrn
Copy link
Contributor

qdrn commented Sep 7, 2022

Endorsements should satisfy the following constraints:

  • Endorsements should be created and propagated to the network by endorsers at half-slot time.
  • Endorsements should endorse the best parent in the given thread as seen by the endorser at this timestamp.
  • Endorsements created at a half-period can only be included in the block of the next period.

Side note: currently when we include the endorsements in a block we verify that they endorse the parent of the created block in this thread.

cc @sebastien-forestier @leoloco

@damip damip added this to the TEST.15.X milestone Sep 9, 2022
@damip
Copy link
Member

damip commented Sep 9, 2022

Current implementation

Currently, we PoS-select endorsement creators targeting the blockclique block at slot (period, thread) if any.
The endorsements selected at (period, thread) are created/emitted at time genesis_timestamp + period * t0 + thread * (t0/thread_count) + t0/2.

Implementation change

We need to PoS-select endorsement creators that are allowed to be included in blocks at slot (period, thread).
The endorsements selected at (period, thread) are created/emitted at time genesis_timestamp + period * t0 + thread * (t0/thread_count) - t0/2 and endorse the latest blockclique (or final) block of their thread.

@sebastien-forestier
Copy link

One additionnal change to be discussed: take into account the compatible endorsements in the pool to compute the cliques fitness

@qdrn
Copy link
Contributor Author

qdrn commented Sep 11, 2022

One additionnal change to be discussed: take into account the compatible endorsements in the pool to compute the cliques fitness

What cases do you have in mind ? If you create a block you create the one that gives the clique with the highest fitness (I guess ?). If you create endorsements then you probably don't have the other endorsements.

@damip
Copy link
Member

damip commented Sep 13, 2022

Part 1 is done here: #3004
Not merging yet (breaking)

@AurelienFT
Copy link
Contributor

Waiting for feeedback from research team.

@AurelienFT AurelienFT removed this from the TEST.15.X milestone Nov 25, 2022
@qdrn
Copy link
Contributor Author

qdrn commented Apr 5, 2023

I think we can close this one. Right ?

@AurelienFT
Copy link
Contributor

take into account the compatible endorsements in the pool to compute the cliques fitness

This hasn't been discuss but could be moved to a discussion

@qdrn
Copy link
Contributor Author

qdrn commented Apr 5, 2023

take into account the compatible endorsements in the pool to compute the cliques fitness

This hasn't been discuss but could be moved to a discussion

Yeah, I don't think it belongs to an issue.

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

No branches or pull requests

5 participants