You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Once the gov param for internal incentives uptime is set up, we should ensure that the logic described in [issue above] is executed properly.
Suggested Design
This component should be quite straightforward to implement due to the decisions made in #7369, as properly handling internal gauge distributions would suffice to achieve the above. Specifically, adding a check to distributeInternal here:
that replaces gaugeTime with the gov param value if it is an internal gauge (& falls back to default if gov param is invalid) should work well. This check should be straightforward to do with logic similar to the following:
Background
Once the gov param for internal incentives uptime is set up, we should ensure that the logic described in [issue above] is executed properly.
Suggested Design
This component should be quite straightforward to implement due to the decisions made in #7369, as properly handling internal gauge distributions would suffice to achieve the above. Specifically, adding a check to
distributeInternal
here:osmosis/x/incentives/keeper/distribute.go
Line 589 in 7c81b90
that replaces
gaugeTime
with the gov param value if it is an internal gauge (& falls back to default if gov param is invalid) should work well. This check should be straightforward to do with logic similar to the following:osmosis/x/incentives/keeper/gauge.go
Line 158 in 7c81b90
Specifically, add the following logic under the first section above:
Testing Strategy
Add the following to existing distributeInternal tests related to CL/NoLock gauges:
Acceptance Criteria
The text was updated successfully, but these errors were encountered: