Skip to content

Conversation

@treeoflife2
Copy link
Member

@llamabutler
Copy link

The rho-trading.ts adapter exports:

> [email protected] test
> ts-node --transpile-only cli/testAdapter.ts open-interest rho-trading.ts

🦙 Running RHO-TRADING.TS adapter 🦙
---------------------------------------------------
Start Date:	Wed, 05 Nov 2025 00:00:00 GMT
End Date:	Thu, 06 Nov 2025 00:00:00 GMT
---------------------------------------------------

ETHEREUM 👇
Open interest at end: 20.22 M

@treeoflife2
Copy link
Member Author

@gephaest can you verify the numbers, i can see you are directly adding the usdc value in the tickers api. does this numbers look accurate to you?

@gephaest
Copy link
Contributor

gephaest commented Nov 7, 2025

@treeoflife2 Hi. In my first version, I added price conversion for better precision.
5ecc674#diff-4d8dbd5f94c0146894abd169742ead3463ee4a431892895b5e112ad91feb7fd1R95

EXCHANGE_INFO_URL provides the actual active pairs, base currencies, and exchange rates. But if you prefer simplicity, stablecoins are close to a 1:1 rate.
On the other hand, I can’t guarantee that we’ll support only stablecoins in the future.

@treeoflife2
Copy link
Member Author

treeoflife2 commented Nov 7, 2025

@treeoflife2 Hi. In my first version, I added price conversion for better precision. 5ecc674#diff-4d8dbd5f94c0146894abd169742ead3463ee4a431892895b5e112ad91feb7fd1R95

EXCHANGE_INFO_URL provides the actual active pairs, base currencies, and exchange rates. But if you prefer simplicity, stablecoins are close to a 1:1 rate. On the other hand, I can’t guarantee that we’ll support only stablecoins in the future.

but you will still show the usd value of OI in exchange UI, so i think this is fine.

@gephaest
Copy link
Contributor

gephaest commented Nov 7, 2025

@treeoflife2 It depends on base currency of future. Now it only USDT/USDC

@treeoflife2 treeoflife2 merged commit 18a77ca into DefiLlama:master Nov 8, 2025
2 checks passed
@treeoflife2 treeoflife2 deleted the rho-x-add-oi branch November 8, 2025 07:24
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