Skip to content

Luxor/reward-curve-proposal#5086

Merged
mnaamani merged 19 commits intoJoystream:luxorfrom
dobertRowneySr:luxor/reward-curve-proposal
Apr 4, 2024
Merged

Luxor/reward-curve-proposal#5086
mnaamani merged 19 commits intoJoystream:luxorfrom
dobertRowneySr:luxor/reward-curve-proposal

Conversation

@dobertRowneySr
Copy link
Copy Markdown
Collaborator

addresses: #3702

@dobertRowneySr dobertRowneySr force-pushed the luxor/reward-curve-proposal branch from f69432c to ee37616 Compare March 13, 2024 11:22
Copy link
Copy Markdown
Collaborator Author

@dobertRowneySr dobertRowneySr left a comment

Choose a reason for hiding this comment

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

I think I still have to add benchmarks for this but at this point I would wait before

  • Council reduce budget
  • Budget WG spending

@kdembler
Copy link
Copy Markdown
Collaborator

kdembler commented Mar 27, 2024

Let's allow the multiplier to be 0.1-2.0. Mokhtar suggests to rename from damping to just multiplier

@kdembler kdembler requested a review from mnaamani March 29, 2024 10:00
@dobertRowneySr dobertRowneySr requested a review from mnaamani April 4, 2024 08:39
Copy link
Copy Markdown
Member

@mnaamani mnaamani left a comment

Choose a reason for hiding this comment

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

Ok LGTM.
Changes to runtime-upgrade integration test do not look complete and they will be addressed separately as per #5109

@mnaamani mnaamani merged commit 3580787 into Joystream:luxor Apr 4, 2024
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.

5 participants