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

Updates #6

Merged
merged 9 commits into from
Apr 28, 2023
Merged

Updates #6

merged 9 commits into from
Apr 28, 2023

Conversation

egunawan85
Copy link

Updates


### Motivation

ICS-101 Interchain Swaps enables chains their own token pricing mechanism and exchange protocol via IBC transactions. Each chain can thus play a role in a fully decentralised exchange network.

Users might also prefer single asset pools over dual assets pools as it removes the risk of impermanent loss.
Features include an option to provide liquidity with a single asset instead of a pair, which users might prefer as it removes the risk of impermanent loss.

Choose a reason for hiding this comment

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

reduce risk

- Timeout: Swap order need to be confirmed on the counterparty chain. it would be canceled and refund if packets not arrival the counterparty on time.
- Slippage Tolerance can be a way to protect loss caused by in-consistency.
- Single side trade: Each Token can only be trade its native chain. in inconsistency state, the backlog swap would sell lower prices than consistency state. which could help to maintain consistency.
- Timeout: Packets timeout when packet acknowledgements are delayed for extended period of times. This limits the impact of inconsistent pool states on mispriced swap orders.

Choose a reason for hiding this comment

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

timeout when packet delivery, acknowledgement don't have timeout,

Copy link

@liangping liangping left a comment

Choose a reason for hiding this comment

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

I left two comments. but I just want to merge it asap to avoid confilct.

@liangping liangping merged commit 13bc90a into sideprotocol:interchain-swap Apr 28, 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