Add Cache.asset_name_to_coin/1 and Cache.asset_name_to_index/1 #3
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
2 collections were introduced in the
Cachemodule:assets_name_to_coin_mapandassets_name_to_index_mapand two new functionsasset_name_to_coin/1andasset_name_to_index/1.assets_name_to_coin_mapmaps perp assets (e.g: BTC, ETH; SOL, etc...) to their respectivecoin(BTC, ETH, SOL, ...) and spot assets (e.g: PURR/USDC, HFUN/USDC, JEFF/USDC, etc...) to their respectivecoin(PURR/USDC, @1, @4, @...)The naming
coinis in line with the official doc https://hyperliquid.gitbook.io/hyperliquid-docs/for-developers/api/info-endpoint#perpetuals-vs-spotassets_name_to_index_mapmaps perp assets (e.g: BTC, ETH; SOL, etc...) to their respective index (0, 1, 5, etc...) and spot assets (e.g: PURR/USDC, HFUN/USDC, JEFF/USDC, etc...) to their respective index (10_000, 10_001, 10_004, etc...)The following modules
lib/hyperliquid/orders/order_wire.exandlib/hyperliquid/orders/orders.exwere also updated to use the new functions.I am open to any discussion on the naming and improvements to the functions.