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

Liquidator: randomly select token/perps for rebalancing #921

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

farnyser
Copy link
Contributor

randomly select token/perps for rebalancing to avoid failing at every try if one token is having an issue

…ing at every try if one token is having an issue
@farnyser farnyser changed the title liquidator: randomly select token/perps for rebalancing Liquidator: randomly select token/perps for rebalancing Mar 25, 2024
@farnyser farnyser merged commit e3a7ed9 into dev Mar 27, 2024
13 checks passed
@farnyser farnyser deleted the serge/liquidator-rebalancing-issue branch March 27, 2024 13:07
}

rebalance_perps_res.expect("rebalancing perps failed");
rebalance_tokens_res.expect("rebalancing tokens failed");
Copy link
Contributor

Choose a reason for hiding this comment

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

this means it'll panic if only the token or only the perp rebalancing fails, I'd prefer to use _res? here

farnyser added a commit that referenced this pull request Apr 18, 2024
…ing at every try if one token is having an issue (#921)

(cherry picked from commit e3a7ed9)
microwavedcola1 pushed a commit that referenced this pull request Apr 22, 2024
* liquidator: randomly select token/perps for rebalancing to avoid failing at every try if one token is having an issue (#921)

(cherry picked from commit e3a7ed9)

* liquidator: do not panic if token or perp rebalancing fails (#927)

(cherry picked from commit 0b7e62e)

* Liquidator: add Sanctum swap (#919)

liquidator: add sanctum swap
(cherry picked from commit 01d5237)

* liquidator: add more LST for sanctum swap (#944)

(cherry picked from commit c0b61b3)
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.

3 participants