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]: Add support for CW pools in group gauge syncing once they have gauge support #6403

Open
AlpinYukseloglu opened this issue Sep 14, 2023 · 2 comments

Comments

@AlpinYukseloglu
Copy link
Contributor

Background

Once CW pools have gauges created post #6122, syncGroupGaugeWeights should be updated to be able to split across them as well.

Suggested Design

  • Add case for CW pools in syncGroupGaugeWeights based on however they are decided to be identified (e.g. maybe a subset of NoLock)

Acceptance Criteria

  • All new and existing tests pass
@AlpinYukseloglu AlpinYukseloglu added the F: volume-splitting feat: Volume-splitting incentives label Sep 14, 2023
@p0mvn
Copy link
Member

p0mvn commented Sep 21, 2023

Ref: #6122 (comment)

cw pool incentives are not well-defined at the moment so we might not need this effort

@p0mvn
Copy link
Member

p0mvn commented Sep 25, 2023

I'm going to move this out of the volume-splitting feature since we decided to avoid pursuing CW pool gauges for v20. Instead, keep this in the icebox

@p0mvn p0mvn added T:ice-box and removed F: volume-splitting feat: Volume-splitting incentives labels Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs Triage 🔍
Development

No branches or pull requests

2 participants