Skip to content

simplicity: activate on first week of unanimous signalling after block 3333333 #1427

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

Merged

Conversation

apoelstra
Copy link
Member

This patch simply starts signalling for Simplicity in any blocksigner that has it enabled. Once we have one week (10080 blocks) of unanimous signalling, then it activates.

Copy link
Member

@delta1 delta1 left a comment

Choose a reason for hiding this comment

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

ACK fa02ff7

confirmed the timestamp/date, and that it's for LiquidV1 params

@psgreco psgreco force-pushed the 2025-03--simplicity-activation branch from fa02ff7 to fcf644e Compare April 14, 2025 11:24
@apoelstra apoelstra force-pushed the 2025-03--simplicity-activation branch from fcf644e to fa02ff7 Compare April 14, 2025 16:19
@apoelstra
Copy link
Member Author

Rebased. (My diff is identical to Pablo's rebase, but mine has a gpg signature from me on it.)

@apoelstra apoelstra force-pushed the 2025-03--simplicity-activation branch from fa02ff7 to 816a39e Compare April 14, 2025 16:19
@delta1
Copy link
Member

delta1 commented Apr 15, 2025

Just remembered that we interpret nStartTime as a block height, not a timestamp

// ELEMENTS: Interpreted as block height!

@delta1
Copy link
Member

delta1 commented Apr 15, 2025

Liquidv1 has passed block 3333333 yesterday which would make a neat/memorable start height

@apoelstra
Copy link
Member Author

Oof, good catch!

@apoelstra apoelstra force-pushed the 2025-03--simplicity-activation branch from 816a39e to 6e1854f Compare April 15, 2025 14:45
…k 3333333

This block passed on April 14 (so we will activate as soon as this is
merged and there is a week of unaminous signalling), and its a nice memorable
number.
@apoelstra apoelstra force-pushed the 2025-03--simplicity-activation branch from 6e1854f to 308a393 Compare April 15, 2025 15:56
Copy link
Member

@delta1 delta1 left a comment

Choose a reason for hiding this comment

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

reACK 308a393

@psgreco psgreco changed the title simplicity: activate on first week of unanimous signalling after March 3 simplicity: activate on first week of unanimous signalling after block 3333333 Apr 15, 2025
@psgreco psgreco merged commit ea5d6fd into ElementsProject:master Apr 15, 2025
11 of 13 checks passed
@apoelstra apoelstra deleted the 2025-03--simplicity-activation branch April 16, 2025 00:49
delta1 added a commit to delta1/elements that referenced this pull request Apr 16, 2025
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

Successfully merging this pull request may close these issues.

3 participants