Skip to content

pull num_uids from subtensor#92

Merged
shibshib merged 2 commits intomainfrom
gating_uid_fix
Jul 13, 2023
Merged

pull num_uids from subtensor#92
shibshib merged 2 commits intomainfrom
gating_uid_fix

Conversation

@ifrit98
Copy link
Contributor

@ifrit98 ifrit98 commented Jul 10, 2023

This eliminates guesswork in determining how many UIDs to initialize the gating model with per subnet.

@p-ferreira
Copy link
Contributor

@Eugene-hu Do you see this as something to be approved asap to accelerate development on net11 (therefore merge to main) or is it something to be pushed in the next release (merge to staging)?

Copy link
Contributor

@p-ferreira p-ferreira left a comment

Choose a reason for hiding this comment

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

LGTM!

@ifrit98 ifrit98 requested a review from p-ferreira July 11, 2023 15:23
@Eugene-hu
Copy link
Contributor

@Eugene-hu Do you see this as something to be approved asap to accelerate development on net11 (therefore merge to main) or is it something to be pushed in the next release (merge to staging)?

It depends on when we are releasing the emissions to netuid 11 and 21. I think this should be released inline with the emission changes, and so the validator can be run without issues. We can do a seperate release for the changes

@shibshib shibshib changed the base branch from main to staging July 13, 2023 15:08
@shibshib shibshib changed the base branch from staging to main July 13, 2023 16:11
@shibshib shibshib merged commit 49d4335 into main Jul 13, 2023
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