Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Conversation

@gilescope
Copy link
Contributor

@gilescope gilescope commented Jul 11, 2022

Reran the weights using the standard CI infrastructure including the new pallets.

@muharem
Copy link
Contributor

muharem commented Jul 11, 2022

@gilescope can you give more context on the PR description.
I see some of weights increased a lot, some of them doubled.

@gilescope
Copy link
Contributor Author

Yeah some of the weights look bigger than expected.

is 3 million where as we're quoting 6 million here. But having bigger weights is just more secure (more expensive too). I suspect they will settle down on future re-runs, but these values are safe. (We will undoubtedly rerun the weights again before the production release so we can keep an eye on the changes.)

@gilescope gilescope added A0-pleasereview B0-silent Changes should not be mentioned in any release notes labels Jul 11, 2022
@joepetrowski joepetrowski merged commit f7a293b into alliance-bench Jul 11, 2022
@joepetrowski joepetrowski deleted the weights-collectives-1427 branch July 11, 2022 16:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

B0-silent Changes should not be mentioned in any release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants