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

[VS Incentives]: Fallback to latest weights if fails to sync #6602

Closed
Tracked by #6057
p0mvn opened this issue Sep 29, 2023 · 0 comments · Fixed by #6606
Closed
Tracked by #6057

[VS Incentives]: Fallback to latest weights if fails to sync #6602

p0mvn opened this issue Sep 29, 2023 · 0 comments · Fixed by #6606
Assignees
Labels
F: volume-splitting feat: Volume-splitting incentives

Comments

@p0mvn
Copy link
Member

p0mvn commented Sep 29, 2023

Background

In the current design, we fail to allocate to all gauges within the group if at least one pool fails to sync volume:

err := k.syncGroupWeights(ctx, group)
if err != nil {
ctx.Logger().Error("error syncing group gauge weights, skipping", "group gauge id", group.GroupGaugeId, "error", err.Error())
continue
}

In that case, incentives stay in the group gauge and get distributed as long as all pools have their volumes updated.

This is undesirable since new pools are likely to have low volume. As a result, all pools associated with such new pools might miss the incentives completely until volume ramps up for all pools.

Additionally, this might cause challenges for external projects incentivizing groups where some of the pools are low-volume

Suggested Design

We should implement the mechanism so that distribution falls back to the old weights if synching fails (as long as they exist)

Acceptance Criteria

  • fallback mechanism implemented
  • tests updated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
F: volume-splitting feat: Volume-splitting incentives
Projects
Archived in project
1 participant