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

feat: Linear map. #2338

Merged
merged 8 commits into from
Oct 31, 2024
Merged

feat: Linear map. #2338

merged 8 commits into from
Oct 31, 2024

Conversation

daniel-wong-dfinity-org
Copy link
Contributor

@daniel-wong-dfinity-org daniel-wong-dfinity-org commented Oct 30, 2024

This was spun out of sns governance so that it can be used elsewhere (in another PR that I'm working on).

References

(No ticket.)

👈 Previous PR | Next PR 👉

@github-actions github-actions bot added the feat label Oct 30, 2024
@daniel-wong-dfinity-org daniel-wong-dfinity-org changed the base branch from master to refresh-voting-power-daniel-wong October 30, 2024 14:46
@daniel-wong-dfinity-org daniel-wong-dfinity-org marked this pull request as ready for review October 30, 2024 14:50
@daniel-wong-dfinity-org daniel-wong-dfinity-org requested a review from a team as a code owner October 30, 2024 14:50
@daniel-wong-dfinity-org daniel-wong-dfinity-org force-pushed the refresh-voting-power-daniel-wong branch from dfd0570 to 58abee2 Compare October 30, 2024 16:12
github-merge-queue bot pushed a commit that referenced this pull request Oct 31, 2024
This occurs when a neuron is modified in the following ways:

1. vote (directly, not indirectly)
2. change follow

# Background

This is part of the "periodic confirmation" feature that was recently
approved in proposal [132411][prop].

[prop]: https://dashboard.internetcomputer.org/proposal/132411

# Future Work

This will be used to reduce the voting power in ballots when a neuron
has not refreshed for a long time.

# References

Closes https://dfinity.atlassian.net/browse/NNS1-3405.

[👈 Previous PR][prev] | [Next PR 👉][next]

[prev]: #2268
[next]: #2338
Base automatically changed from refresh-voting-power-daniel-wong to master October 31, 2024 18:34
@daniel-wong-dfinity-org daniel-wong-dfinity-org added this pull request to the merge queue Oct 31, 2024
Merged via the queue into master with commit 0346088 Oct 31, 2024
24 checks passed
@daniel-wong-dfinity-org daniel-wong-dfinity-org deleted the linear-map-daniel-wong branch October 31, 2024 21:50
github-merge-queue bot pushed a commit that referenced this pull request Nov 5, 2024
…er. (#2339)

Add deciding_voting_power method to Neuron.

Renames the voting_power method of Neuron to potential_voting_power.

Added a flag that controls whether deciding_voting_power applies
adjustment. Per usual, adjustment is off in vanilla builds, but on when
feature = "test".

Voting rewards uses this new method. As long as the the new behavior is
not enabled (via the flag), this should result in no behavior change,
and even initially, when the feature is enabled, since no neuron can be
too unrefreshed, even then, there would be no immediate behavior change.

# Background

This is part of the "periodic confirmation" feature that was recently
approved in proposal [132411][prop].

[prop]: https://dashboard.internetcomputer.org/proposal/132411

# References

Closes https://dfinity.atlassian.net/browse/NNS1-3406,
https://dfinity.atlassian.net/browse/NNS1-3407.

[👈 Previous PR][prev] | [Next PR 👉][next]

[prev]: #2338
[next]: #2375
alin-at-dfinity pushed a commit that referenced this pull request Nov 7, 2024
…er. (#2339)

Add deciding_voting_power method to Neuron.

Renames the voting_power method of Neuron to potential_voting_power.

Added a flag that controls whether deciding_voting_power applies
adjustment. Per usual, adjustment is off in vanilla builds, but on when
feature = "test".

Voting rewards uses this new method. As long as the the new behavior is
not enabled (via the flag), this should result in no behavior change,
and even initially, when the feature is enabled, since no neuron can be
too unrefreshed, even then, there would be no immediate behavior change.

# Background

This is part of the "periodic confirmation" feature that was recently
approved in proposal [132411][prop].

[prop]: https://dashboard.internetcomputer.org/proposal/132411

# References

Closes https://dfinity.atlassian.net/browse/NNS1-3406,
https://dfinity.atlassian.net/browse/NNS1-3407.

[👈 Previous PR][prev] | [Next PR 👉][next]

[prev]: #2338
[next]: #2375
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants