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

Hotfixes to testnet nov 6 #941

Merged
merged 6 commits into from
Nov 7, 2024
Merged

Hotfixes to testnet nov 6 #941

merged 6 commits into from
Nov 7, 2024

Conversation

keithtensor
Copy link
Contributor

Refs #938.

Contains the following PRs to merge into testnet:

* skip netuid 0 for pending emission accum

* comment
* add comment also

* chore: fmt

* add back delta handling in helpers

* track removals in stake delta

* handle stake delta in ck swap func

* add tests back

* add tests for staking back

* add back test for ck swap
* respect chk for set weights min stake filter

* use for test func

* respect chk in set weights call also

* add tests back

* fix weights min stake tests also
@keithtensor keithtensor requested a review from a team November 7, 2024 09:08
@keithtensor keithtensor requested a review from unconst as a code owner November 7, 2024 09:08
@keithtensor keithtensor added skip-cargo-audit This PR fails cargo audit but needs to be merged anyway and removed skip-cargo-audit This PR fails cargo audit but needs to be merged anyway labels Nov 7, 2024
gztensor
gztensor previously approved these changes Nov 7, 2024
camfairchild
camfairchild previously approved these changes Nov 7, 2024
Copy link
Contributor

@camfairchild camfairchild left a comment

Choose a reason for hiding this comment

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

Don't forget to bump the spec

@gztensor gztensor dismissed stale reviews from camfairchild and themself via fd7a996 November 7, 2024 18:46
@gztensor gztensor added the skip-cargo-audit This PR fails cargo audit but needs to be merged anyway label Nov 7, 2024
@unconst unconst merged commit 05a53d7 into testnet Nov 7, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-cargo-audit This PR fails cargo audit but needs to be merged anyway
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants