Skip to content

[release] v0.8.0#586

Merged
ca333 merged 297 commits intomasterfrom
dev
Jun 8, 2023
Merged

[release] v0.8.0#586
ca333 merged 297 commits intomasterfrom
dev

Conversation

@ca333
Copy link
Copy Markdown

@ca333 ca333 commented May 30, 2023

changelog placehodler

@ca333 ca333 requested review from Alrighttt, DeckerSU, dimxy and smk762 May 30, 2023 06:41
@dimxy
Copy link
Copy Markdown
Collaborator

dimxy commented May 30, 2023

There is a test TestNotary.KomodoNotaries which needs to be updated for the new season (it checks new timestamp and pubkeys).
I will add it in a new branch. Do we merge it now or later after the hardfork?

@DeckerSU
Copy link
Copy Markdown

Seems we missed #582. We should include it before HF, i guess, as we already mentioned hardware requirements for NNs based on Equihash solutions memory optimizations, introduced in #582. This PR also includes additional error checking and performance optimizations, like boost node startup time, etc.

dimxy and others added 3 commits May 30, 2023 22:47
Copy link
Copy Markdown
Collaborator

@dimxy dimxy left a comment

Choose a reason for hiding this comment

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

I have tested the new interest calculation, fixed tests for getting S7 pubkeys, all tests now work.
So it LGTM.
Probably we are short of time for publishing this release so let's do this ASAP.
I will continue testing though

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. The tests for HF simulation, among others, were passed successfully, making the product ready for use on the mainnet.

p.s. I will also keep carrying out tests on the HF simulation network.

@ca333 ca333 merged commit 156dba6 into master Jun 8, 2023
who-biz pushed a commit to who-biz/komodo that referenced this pull request Jun 22, 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.

7 participants