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

Invalid Price Cache #164

Open
TheDenverWest opened this issue Apr 3, 2022 · 9 comments
Open

Invalid Price Cache #164

TheDenverWest opened this issue Apr 3, 2022 · 9 comments

Comments

@TheDenverWest
Copy link

TheDenverWest commented Apr 3, 2022

Problem statement:
As a trader on the mangos DEX I need to be able to place orders into the book timely so that I can protect my capital and earn profits to pay the DEX and increase revenue for mango holders.

Impact
This issue is making it borderline impossible to place orders or perform other actions on mangos v3. Even when Solana network is hovering above 1700TPS. Other Dapps/smart contracts on Solana are functioning flawlessly at these speeds.
image
image

Anecdotally: I have tried over 200 times to write a SOL-PERP order ( of type Stop Loss ) while Solana is hovering above 1700 TPS (4/3/22 4-5AM MST) and haven't gotten a single order through in the last hour. TOW there are multiple people complaining about the issue in the discord. It is becoming a common theme which may be costing mangos holders.

Dev considerations

  1. Research into why pyth/Switchboard are having such an issue inserting price data into blocks.
    image

  2. Research an estimated Solana Txn/s speed above which it will be safe to use Mangos.
    I suggest implementing a warning or some sort of user facing message stating orders have a very high likelihood of failing to confirm below XXXXTPS. This will ensure users are educated about the REAL risk they may be taking on, at least until the research items (1 & 2 ) can be completed.

  3. From: IwillNotSaveYou On what the issue may be.
    The bottom line is there's probably some asset whose oracle is bad (i'm not sure which one), and if you hold that asset in your account it effectively siezes the whole account.

Acceptance Criteria:
The issue in item 1 is fully understood and the partners pyth/switchboard are aware and incentivized to work with mango on a resolution.
Users should be able to reliably place and remove orders, deposit and withdraw assets on mangos.v3 when Solana TPS are above the TPS researched in item 2.

@TheDenverWest
Copy link
Author

TheDenverWest commented Apr 3, 2022

User: KenM#5586
Date: 4/3 5AM MST
Asset: SOL-Perp Order Type: Stop-Limit
Asset: LUNA/USDC Action: BUY
RPC: Triton/Genesys/Serum

image

@TheDenverWest
Copy link
Author

TheDenverWest commented Apr 3, 2022

User: Ki#9265
Date: 4/3 5AM MST
Asset: SOL-Perp
Order Type: Stop-Limit
RPC: Triton/Genesys/Serum
image

@TheDenverWest
Copy link
Author

TheDenverWest commented Apr 3, 2022

User: Ace10Suted#4114
Date: 4/2 10PM MST
Asset: SRM
Action Type: Withdraw
image

@TheDenverWest
Copy link
Author

TheDenverWest commented Apr 3, 2022

User: ToreroRomero#1584
Date: 4/3 5AM MST
Asset:
Action: Withdraw
RPC:
image

@TheDenverWest
Copy link
Author

TheDenverWest commented Apr 3, 2022

User: clevelandasd#5827
Date: 4/3 6AM MST
Asset: BTC-PERP
Action: Limit Order [buy + sell]
RPC: https://mango.rpcpool.com/946ef7337da3f5b8d3e4a34e7f88
image

@TheDenverWest
Copy link
Author

TheDenverWest commented Apr 3, 2022

INTERESTING:

This account is getting many orders through, even though almost all other users are unable to do any actions at TOW.(withdraw/deposit/place orders/ etc.)
https://trade.mango.markets/account?pubkey=DUp1NPypVm32hniFLY7KqCRtNQ6dM7ipQbTfuXh48gDt

@TheDenverWest
Copy link
Author

TheDenverWest commented Apr 3, 2022

User: 💎.sol#5896
Date: 4/3 6AM MST
Asset:
Action: Market Close
RPC: Triton
image

@TheDenverWest
Copy link
Author

TheDenverWest commented Apr 3, 2022

User: Harm#8517
Date: 4/3 6AM MST
Asset: SOL-PERP
Action: Stop Loss
RPC: Triton/Genesys/Serum
image

@TheDenverWest
Copy link
Author

User: Rando2020#5684
Date: 4/3 6AM MST
Asset:
Action: Withdraw
RPC: Triton/Genesys/Serum
image

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

No branches or pull requests

1 participant