Skip to content
This repository was archived by the owner on Apr 14, 2026. It is now read-only.

integrate votemarket vebal/vlaura data into allocator automation - #302

Merged
jalbrekt85 merged 4 commits into
biweekly-runsfrom
fix/aura-split-and-beets-transfer
Mar 11, 2026
Merged

integrate votemarket vebal/vlaura data into allocator automation#302
jalbrekt85 merged 4 commits into
biweekly-runsfrom
fix/aura-split-and-beets-transfer

Conversation

@jalbrekt85

@jalbrekt85 jalbrekt85 commented Mar 6, 2026

Copy link
Copy Markdown
Collaborator

solves #295 - problem is that data cant be queried for a round until that round ends. solution is new automation, backfill_aura_split.yaml, that runs on after a new rounds ends and backfills the data automatically. backfilled data starting from fee runs after 1767225600 have been added here too

also fixes #300

Comment thread fee_allocator/votemarket_analytics.py Outdated
data = _fetch_json(round_url_template.format(round_id=rid))
for gauge in data["analytics"]:
addr = gauge["gauge"].lower()
deposited[addr] = deposited.get(addr, 0) + gauge["totalDeposited"]

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Xeonus suggests to use nonBlacklistedVotes instead of totalDeposited in #295, not sure if there is a difference?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

did a deep dive and this is a big bug:

get_aura_share_gauge should be based on the votes, not the deposit

i think this is also why we are seeing unexpected low numbers for the backfilled auravebalShare (<=.5), whereas i would expect the usual ~.65

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

good catch. yes, should be using nonBlacklistedVotes. fixed and updated backfilled data too: 76e038

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Agreed! Great find and fix looks good

Comment on lines +37 to +38
bal_round_ids = _find_matching_rounds(bal_metadata, period_start, period_end)
aura_round_ids = _find_matching_rounds(aura_metadata, period_start, period_end)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

should we check if there is actually data? not sure on the cron timing and the data availability on stakedao's side

@jalbrekt85 jalbrekt85 Mar 10, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

good point. added to check to handle this scenario better. if no data, it will just exit early and new data will be caught the following week: 76e038

@Xeonus Xeonus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Fixes issues previously found with backfilling of data. Uses suggested approach from #295

@jalbrekt85
jalbrekt85 merged commit abc67aa into biweekly-runs Mar 11, 2026
4 checks passed
@jalbrekt85
jalbrekt85 deleted the fix/aura-split-and-beets-transfer branch March 11, 2026 19:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Payload generates negative USDC transfer for Beets when fees are zero

3 participants