Skip to content

Fix Protocol Liquidation Fee calculation #313

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 31, 2023
Merged

Fix Protocol Liquidation Fee calculation #313

merged 1 commit into from
Aug 31, 2023

Conversation

piobab
Copy link
Collaborator

@piobab piobab commented Aug 30, 2023 •
edited
Loading

Should be:

debt_value_to_repay = debt_amount_to_repay * debt_2_price

collateral_amount_to_liquidate = debt_value_to_repay * (1 + LB) / collateral_1_price

protocol_fee_value = debt_value_to_repay * LB * PLF

protocol_fee_amount = protocol_fee_value / collateral_1_price

collateral_amount_received_by_liquidator = collateral_amount_to_liquidate - protocol_fee_amount

@piobab piobab force-pushed the fix/plf-adjustment-for-dynamic-liq branch from aff8b1f to 7b50b75 Compare August 30, 2023 09:16
@piobab piobab force-pushed the fix/plf-adjustment-for-dynamic-liq branch from 7b50b75 to e3ef0f6 Compare August 30, 2023 09:44
@piobab piobab requested review from brimigs, dancreee and thec00n August 30, 2023 10:21
@piobab piobab merged commit 8c90842 into release/mars-v2 Aug 31, 2023
@piobab piobab deleted the fix/plf-adjustment-for-dynamic-liq branch September 14, 2023 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dancreee dancreee dancreee approved these changes

@thec00n thec00n Awaiting requested review from thec00n

+1 more reviewer

@brimigs brimigs brimigs approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@piobab @dancreee @brimigs