Skip to content

Update season 5 pubkeys and constants#424

Merged
ca333 merged 8 commits intodevfrom
smk762-patch-S5-consts
May 9, 2021
Merged

Update season 5 pubkeys and constants#424
ca333 merged 8 commits intodevfrom
smk762-patch-S5-consts

Conversation

@smk762
Copy link
Copy Markdown

@smk762 smk762 commented May 8, 2021

No description provided.

@smk762 smk762 requested review from Alrighttt, DeckerSU and ca333 May 8, 2021 00:06
@Alrighttt
Copy link
Copy Markdown

Looks good. I will coordinate a hard fork testnet in the coming week using this code.

Alrighttt
Alrighttt previously approved these changes May 9, 2021
const int32_t nS4HardforkHeight = 1922000; //dPoW Season 4 2020 hardfork Sunday, June 14th, 2020

const uint32_t nS5Timestamp = 1623682800; //dPoW Season 5 Monday, June 14th, 2021 (03:00:00 PM UTC)
const int32_t nS5HardforkHeight = 2437300; //dPoW Season 5 Monday, June 14th, 2021
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

2437300 - 2384484 = 52816
52816/1440 = ~36.6 days from now

Probably ok, but likely to happen earlier than the comment states

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Using height_in_future = last_block_number + ((future_time - time_now) / block_time)
2384540 + ((1623682800 - 1620522112)/60) = 2437218.133333333

Can update just before merge to get closer value if needed, seems this will shift depending on block production rate in the interval.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

2437300 - 2384484 = 52816
52816/1440 = ~36.6 days from now

Probably ok, but likely to happen earlier than the comment states

image

DeckerSU
DeckerSU previously approved these changes May 9, 2021
Copy link
Copy Markdown

@DeckerSU DeckerSU left a comment

Choose a reason for hiding this comment

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

LGTM, but seems we forgot to change NUM_KMD_SEASONS constant here:
https://github.com/KomodoPlatform/komodo/blob/fee6209f729c4b3730d348be99346dfeb0d0ca7c/src/komodo_defs.h#L49
Should be 6.

ca333
ca333 previously approved these changes May 9, 2021
@DeckerSU DeckerSU self-requested a review May 9, 2021 01:07
fix num seasons
@smk762 smk762 dismissed stale reviews from ca333, DeckerSU, and Alrighttt via 3da453a May 9, 2021 01:09
@ca333 ca333 merged commit 48a9540 into dev May 9, 2021
DeckerSU added a commit to DeckerSU/KomodoOcean that referenced this pull request May 16, 2021
who-biz pushed a commit to who-biz/komodo that referenced this pull request Jun 14, 2022
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.

5 participants