Skip to content

Increase count limit#101

Merged
Eugene-hu merged 23 commits intostagingfrom
count_increase_limit
Jul 24, 2023
Merged

Increase count limit#101
Eugene-hu merged 23 commits intostagingfrom
count_increase_limit

Conversation

@Eugene-hu
Copy link
Contributor

  • Increases the base count limit to 3000. By increasing the history, the change decreases the variance observed by the miners due to fluctuations in data. This also ensures that any outlier in both the prompt dataset and answers have less effect on future responses
  • Currently, at the limit of 1000 responses, we observe fluctuations are heavily impacted by outliers, which has lasting effects and increases the variance in rewards. This change will decrease the effects of outliers by 3x.
  • 3000 responses at 80% response rate can be achieved most validators within 200-300 blocks; well within the expected lifetime of the validator. If we were to further increase the count limit, a saving mechanism should be implemented for the reward models to enable faster consensus.

image

@Eugene-hu Eugene-hu marked this pull request as ready for review July 20, 2023 22:54
@mrseeker
Copy link
Contributor

mrseeker commented Jul 23, 2023

Why was this approved and then reverted?

EDIT: e422d2a

@Eugene-hu Eugene-hu changed the base branch from main to staging July 24, 2023 15:40
@Eugene-hu
Copy link
Contributor Author

Why was this approved and then reverted?

EDIT: e422d2a

This was an accident, we are going to release this change in the next release.

@Eugene-hu Eugene-hu merged commit 4fc3a9c into staging Jul 24, 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.

6 participants