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.

Sign up for GitHub

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

Jump to bottom

[Incentives Module][bugfix] Scale gas costs by denoms in gauge #4738

Closed
AlpinYukseloglu opened this issue Mar 24, 2023 · 0 comments
Closed

[Incentives Module][bugfix] Scale gas costs by denoms in gauge #4738

AlpinYukseloglu opened this issue Mar 24, 2023 · 0 comments
Assignees
@stackman27
Labels
F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board

Comments

@AlpinYukseloglu
Copy link
Contributor

Background

Similarly to what we did for incentive records in CL, we should be scaling gas costs for adding to gauges to prevent spam, as distribution logic linearly iterates over all denoms in a gauge at epoch.

Suggested Design

  • Add scaling gas costs to AddToGaugeRewards, similar to how we did it for incentive records

Acceptance Criteria

  • New and existing tests pass
@AlpinYukseloglu AlpinYukseloglu added the F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board label Mar 24, 2023
@github-project-automation github-project-automation bot moved this to Needs Triage 🔍 in Osmosis Chain Development Mar 24, 2023
@AlpinYukseloglu AlpinYukseloglu moved this from Needs Triage 🔍 to Todo 🕒 in Osmosis Chain Development Mar 25, 2023
@stackman27 stackman27 self-assigned this Mar 26, 2023
@stackman27 stackman27 mentioned this issue Apr 4, 2023
Merged
@p0mvn p0mvn moved this from Todo 🕒 to Needs PR Review in Osmosis Chain Development Apr 25, 2023
@p0mvn p0mvn closed this as completed Apr 25, 2023
@github-project-automation github-project-automation bot moved this from Needs PR Review to Done ✅ in Osmosis Chain Development Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stackman27 stackman27

Labels
F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board
Projects
Osmosis Chain Development
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@stackman27 @p0mvn @AlpinYukseloglu