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

refactor(nns): Refactor voting as precursor for more scalability #2528

Merged
merged 11 commits into from
Nov 15, 2024

Conversation

max-dfinity
Copy link
Contributor

@max-dfinity max-dfinity commented Nov 9, 2024

This refactoring creates a state machine to handle voting, which will create natural break points in the ongoing calculations that are being done. This will allow us to scale voting with following to an arbitrarily large number of neurons.

Next

@max-dfinity max-dfinity force-pushed the msum/stable-neuron-support-C branch from 494f084 to 9cdf93c Compare November 11, 2024 19:15
@max-dfinity max-dfinity changed the title refactor(nns): Refactor voting to allow for multi-message support refactor(nns): Refactor voting as precursor for more scalability Nov 12, 2024
@max-dfinity max-dfinity force-pushed the msum/stable-neuron-support-C branch 2 times, most recently from 406f631 to d2eb45e Compare November 13, 2024 00:50
@max-dfinity max-dfinity force-pushed the msum/stable-neuron-support-C branch 2 times, most recently from a08f2cc to e571398 Compare November 13, 2024 18:58
@max-dfinity max-dfinity force-pushed the msum/stable-neuron-support-C branch from e571398 to 6ec99a1 Compare November 13, 2024 19:25
@max-dfinity max-dfinity marked this pull request as ready for review November 13, 2024 19:47
@max-dfinity max-dfinity requested a review from a team as a code owner November 13, 2024 19:47
rs/nns/governance/src/voting.rs Outdated Show resolved Hide resolved
rs/nns/governance/src/voting.rs Outdated Show resolved Hide resolved
rs/nns/governance/src/voting.rs Outdated Show resolved Hide resolved
rs/nns/governance/src/voting.rs Outdated Show resolved Hide resolved
@max-dfinity max-dfinity force-pushed the msum/stable-neuron-support-C branch from 8d3213d to 1832771 Compare November 15, 2024 02:11
rs/nns/governance/src/voting.rs Outdated Show resolved Hide resolved
rs/nns/governance/src/voting.rs Show resolved Hide resolved
rs/nns/governance/src/voting.rs Outdated Show resolved Hide resolved
@max-dfinity max-dfinity force-pushed the msum/stable-neuron-support-C branch 2 times, most recently from 2cb808a to 886676d Compare November 15, 2024 20:49
@max-dfinity max-dfinity disabled auto-merge November 15, 2024 20:57
@max-dfinity max-dfinity force-pushed the msum/stable-neuron-support-C branch from 886676d to 9828e39 Compare November 15, 2024 20:57
@max-dfinity max-dfinity added this pull request to the merge queue Nov 15, 2024
Merged via the queue into master with commit 139b0f2 Nov 15, 2024
24 checks passed
@max-dfinity max-dfinity deleted the msum/stable-neuron-support-C branch November 15, 2024 21:51
@andrew-lee-work
Copy link
Contributor

LGTM

michael-weigelt pushed a commit that referenced this pull request Nov 19, 2024
…ct (#2600)

This puts cast_vote_and_cascade_follow into governance, which will be
necessary for the next step of making it async, and then finally adding
break points to increase the voting capacity.

[Prev](#2528) | Next
DFINITYManu pushed a commit that referenced this pull request Nov 19, 2024
This refactoring creates a state machine to handle voting, which will
create natural break points in the ongoing calculations that are being
done. This will allow us to scale voting with following to an
arbitrarily large number of neurons.

[Next](#2600)
DFINITYManu pushed a commit that referenced this pull request Nov 19, 2024
…ct (#2600)

This puts cast_vote_and_cascade_follow into governance, which will be
necessary for the next step of making it async, and then finally adding
break points to increase the voting capacity.

[Prev](#2528) | Next
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants