Skip to content

scripts: adjust DLLR lm rewards#474

Merged
cwsnt merged 4 commits intodevelopmentfrom
scripts/update_lm_rewards_dllr
Jan 18, 2023
Merged

scripts: adjust DLLR lm rewards#474
cwsnt merged 4 commits intodevelopmentfrom
scripts/update_lm_rewards_dllr

Conversation

@cwsnt
Copy link
Copy Markdown
Contributor

@cwsnt cwsnt commented Jan 15, 2023

Set the AMM Pool WRBTC/DLLR LM rewards to 25k SOV in testnet.
Note: current WRBTC/XUSD LM reward is set to 1 SOV already

Comment thread scripts/deployment/liquidity-mining/update-lm.py Outdated
[contracts['iXUSD'], contracts['LiquidityMiningConfigToken']],
[ALLOCATION_POINT_I_XUSD, ALLOCATION_POINT_CONFIG_TOKEN],
[contracts['(WR)BTC/DLLR'], contracts['LiquidityMiningConfigToken']],
[ALLOCATION_POINT_BTC_DLLR, ALLOCATION_POINT_CONFIG_TOKEN],
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.

missing ALLOCATION_POINT_BTC_XUSD
afaik it is not 1 already, only iXUSD is, but not the AMM pool token

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually, i have checked the allocation point for the AMM pool token (BTC/XUSD) which is 1

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.

this is the poolinfo i get for (W)RBTC/XUSD : ('0x6f96096687952349DD5944E0EB1Be327DcdeB705', 25000, 4975556, 5655368276681260)
allocation points = 25000

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah ya, you are right, was seeing the testnet one.
Updated

@cwsnt cwsnt requested a review from ororopickpocket January 17, 2023 14:07
@cwsnt cwsnt merged commit 591ed38 into development Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants