chore: Add EIGEN,BOME as isolatd market in default test genesis #2651
Conversation
WalkthroughThe pull request introduces modifications to several scripts and configuration files related to the genesis setup of a blockchain protocol. Key changes include enhanced logging in the Changes
Possibly related PRs
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
teddyding
changed the title
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Outside diff range and nitpick comments (1)
protocol/scripts/genesis/prod_pregenesis.sh (1)
171-174: Correct the typographical error in the log messageThere is a typo in the echo statement on line 174. The word "Overwriting" is misspelled as "Oerwriting."
Apply this diff to correct the typo:
- echo "Oerwriting genesis params for production..." + echo "Overwriting genesis params for production..."
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
protocol/scripts/genesis/prod_pregenesis.sh(1 hunks)protocol/scripts/genesis/sample_pregenesis.json(7 hunks)protocol/testing/genesis.sh(4 hunks)protocol/x/clob/types/genesis.go(0 hunks)protocol/x/clob/types/genesis_test.go(0 hunks)protocol/x/perpetuals/types/genesis.go(1 hunks)protocol/x/perpetuals/types/genesis_test.go(0 hunks)
💤 Files with no reviewable changes (3)
- protocol/x/clob/types/genesis.go
- protocol/x/clob/types/genesis_test.go
- protocol/x/perpetuals/types/genesis_test.go
🔇 Additional comments (8)
protocol/testing/genesis.sh (4)
1274-1310: Ensure accurate configuration of market mappings for new assets
The market mappings for EIGEN-USD and BOME-USD are added. Please verify:
- The
decimalsare set appropriately ('9'forEIGEN-USDand'12'forBOME-USD). - The
min_provider_countis correctly set to'1'. - The
provider_configsinclude accurateoff_chain_tickervalues and normalization parameters. - The
metadata_JSONcontains valid JSON with correct market metadata.
1718-1741: Confirm the consistency of price parameters for new markets
New price parameters and initial prices are set for EIGEN-USD (ID 300) and BOME-USD (ID 301). Please ensure:
- The
exponentvalues ('-9'forEIGEN-USDand'-12'forBOME-USD) are appropriate for the assets' price scales. - The
min_price_change_ppmis set to800(0.08%), which differs from the standard 1000 or 2500 ppm. Verify that this deviation is intentional. - The initial
pricevalues ('4973000000'forEIGEN-USDand'8695478191'forBOME-USD) accurately reflect the current market prices multiplied by10^9and10^12respectively, based on the exponents.
2108-2125: Validate the CLOB pair configurations for new markets
The CLOB pairs for EIGEN-USD (ID 300) and BOME-USD (ID 301) are added. Please confirm:
- The
statusis set to the intended initial state ("$INITIAL_CLOB_PAIR_STATUS"). - The
perpetual_idmatches the IDs assigned in the perpetuals section. - The
step_base_quantums,subticks_per_tick, andquantum_conversion_exponentare appropriate for the assets' trading increments and precision requirements.
545-564: Verify the uniqueness and correctness of the new perpetual IDs
The new perpetual markets EIGEN-USD and BOME-USD are assigned IDs 300 and 301 respectively. Ensure that these IDs are unique across the perpetuals.perpetuals array and do not conflict with existing IDs. Also, verify that these IDs are correctly referenced in other parts of the genesis configuration.
Run the following script to check for duplicate perpetual IDs:
✅ Verification successful
Perpetual IDs 300 and 301 are unique and correctly configured
Based on the comprehensive output, I can verify that:
- The perpetual IDs 300 (EIGEN-USD) and 301 (BOME-USD) are unique in the genesis configuration. The existing perpetuals use IDs 0-32, making 300 and 301 non-conflicting.
- The market IDs match their respective perpetual IDs (300 for EIGEN-USD and 301 for BOME-USD).
- The tickers are properly configured in both the perpetuals section and the marketmap section with their corresponding currency pairs and settings.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check for duplicate perpetual IDs in the genesis configuration.
# Extract all perpetual IDs and check for duplicates
grep -oP "\.perpetuals\.\[\d+\]\.params\.id' -v '\K\d+" ./protocol/testing/genesis.sh | sort | uniq -d
Length of output: 110
Script:
#!/bin/bash
# Let's try a different approach to extract and check perpetual IDs
# First, let's get all lines containing perpetual IDs and market IDs
# Get all perpetual ID assignments
echo "=== Perpetual IDs ==="
grep -n "perpetuals\.perpetuals\.[[:digit:]]\+\.params\.id.*-v" protocol/testing/genesis.sh
# Get all market ID assignments
echo -e "\n=== Market IDs ==="
grep -n "perpetuals\.perpetuals\.[[:digit:]]\+\.params\.market_id.*-v" protocol/testing/genesis.sh
# Get all ticker assignments for context
echo -e "\n=== Tickers ==="
grep -n "perpetuals\.perpetuals\.[[:digit:]]\+\.params\.ticker.*-v" protocol/testing/genesis.sh
Length of output: 445
Script:
#!/bin/bash
# Let's try a simpler approach using single quotes and basic pattern matching
echo "=== Perpetual IDs ==="
grep -n "params.id.*'.*'" protocol/testing/genesis.sh
echo -e "\n=== Market IDs ==="
grep -n "market_id.*'.*'" protocol/testing/genesis.sh
echo -e "\n=== Tickers ==="
grep -n "ticker.*'.*'" protocol/testing/genesis.sh
Length of output: 34702
protocol/x/perpetuals/types/genesis.go (1)
48-48: Confirm the removal of sequential ID validation for perpetuals
The sequential ID check for perpetuals has been removed. Now, the validation only ensures that perpetual IDs are unique and that the Ticker field is non-empty. This change accommodates non-sequential IDs like 300 and 301 for the new markets.
Ensure that all components of the system that interact with perpetual IDs are compatible with non-sequential IDs and do not assume sequential ordering.
protocol/scripts/genesis/sample_pregenesis.json (3)
4093-4093: LGTM: Version update is appropriate
The app version update to "7.0.0-dev0-149-g12cfb908b" is consistent with the addition of new markets.
3757-3770:
Review price parameter configurations
Several unique configurations need attention:
- Both markets use min_price_change_ppm: 800, lower than other markets (1000-4000)
- Both introduce a new parameter min_exchanges: 1
- BOME's exponent (-12) is unusually low
Consider aligning these parameters with other markets:
{
"exponent": -12,
"id": 301,
- "min_exchanges": 1,
- "min_price_change_ppm": 800,
+ "min_price_change_ppm": 1000,
"pair": "BOME-USD"
}478-497: Verify the large gap in CLOB pair IDs
The new CLOB pairs use IDs 300 and 301, while existing pairs use 0-32. While this might be intentional for isolated markets, we should verify this design choice.
✅ Verification successful
Let me gather more context about the CLOB pair IDs to understand if this gap pattern is documented or follows a specific design.
Let me check if there's any documentation or configuration that explains the CLOB pair ID ranges.
From the search results, I can see references to isolated markets and their implementation in the codebase. Let me check the genesis configuration to understand the ID ranges better.
From the search results, I can see that IDs 300 and 301 are specifically for isolated markets (market_type: 2), while IDs 0-32 are for cross-margin markets (market_type: 1). Let me verify the market types in the codebase.
Based on the search results, I can see that IDs 300 and 301 are specifically designated for isolated markets (market_type: 2), while IDs 0-32 are used for cross-margin markets (market_type: 1). This is an intentional design choice to separate isolated markets from cross-margin markets using different ID ranges.
The gap in CLOB pair IDs is intentional to distinguish isolated markets (300+) from cross-margin markets (0-32)
The codebase shows clear evidence that:
- Markets with IDs 0-32 are configured as
PERPETUAL_MARKET_TYPE_CROSS - Markets with IDs 300+ are configured as
PERPETUAL_MARKET_TYPE_ISOLATED - This separation in ID ranges helps in easily identifying and managing different market types
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check if there are any CLOB pairs with IDs between 32 and 300
# to ensure we're not missing any intermediate IDs
rg -U "\"id\": \d{2,3}[^0]" --json | jq -r '.data.lines.text' | sort -n | \
awk 'BEGIN{prev=0} {curr=$2+0; if(curr-prev>1) printf "Gap found between IDs %d and %d\n", prev, curr; prev=curr}'
Length of output: 283
Script:
#!/bin/bash
# Check documentation and configuration files for any mentions of CLOB pair ID ranges or patterns
rg -i "clob.*id|market.*id" README.md docs/ protocol/ -A 3 -B 3
# Also check for any comments or documentation near the CLOB definitions
rg -U "\"id\":\s*(300|301)" -B 10
Length of output: 79701
Script:
#!/bin/bash
# Search for documentation about CLOB pair ID ranges or market types
rg -i "clob.*id.*range|market.*id.*range|isolated.*market" -A 3 -B 3 protocol/
# Also check for any comments near the CLOB pair definitions in genesis files
rg -U "\"id\":\s*\d+" protocol/scripts/genesis/ -B 5 -A 5
Length of output: 71681
Script:
#!/bin/bash
# Search for CLOB pair definitions and any comments about ID ranges in genesis files
rg -U "\"id\":|\"perpetual_id\":|\"market_id\":" protocol/scripts/genesis/ -B 2 -A 2
# Also check for any documentation about market types and ID ranges
rg -i "market.*type|isolated.*market|cross.*market" protocol/docs/ protocol/README.md -A 3 -B 3
Length of output: 65769
Script:
#!/bin/bash
# Search for market type definitions and documentation
rg -U "PERPETUAL_MARKET_TYPE|market_type.*=.*2|market_type.*isolated" protocol/ -B 3 -A 3
# Also check for any comments or documentation about market ID ranges
rg -i "market.*id.*range|clob.*id.*range|isolated.*id" protocol/ -B 3 -A 3
Length of output: 70740
| "EIGEN/USD": { | ||
| "provider_configs": [ | ||
| { | ||
| "invert": false, | ||
| "name": "okx_ws", | ||
| "normalize_by_pair": { | ||
| "Base": "USDT", | ||
| "Quote": "USD" | ||
| }, | ||
| "off_chain_ticker": "EIGEN-USDT" | ||
| }, | ||
| { | ||
| "invert": false, | ||
| "name": "bybit_ws", | ||
| "normalize_by_pair": { | ||
| "Base": "USDT", | ||
| "Quote": "USD" | ||
| }, | ||
| "off_chain_ticker": "EIGENUSDT" | ||
| }, | ||
| { | ||
| "name": "crypto_dot_com_ws", | ||
| "off_chain_ticker": "EIGEN_USD" | ||
| }, | ||
| { | ||
| "name": "coinbase_ws", | ||
| "off_chain_ticker": "EIGEN-USD" | ||
| }, | ||
| { | ||
| "name": "kraken_api", | ||
| "off_chain_ticker": "EIGENUSD" | ||
| } | ||
| ], | ||
| "ticker": { | ||
| "currency_pair": { | ||
| "Base": "EIGEN", | ||
| "Quote": "USD" | ||
| }, | ||
| "decimals": 9, | ||
| "enabled": true, | ||
| "metadata_JSON": "{\"reference_price\":3648941500,\"liquidity\":3099304,\"aggregate_ids\":[{\"venue\":\"coinmarketcap\",\"ID\":\"30494\"}]}", | ||
| "min_provider_count": 1 | ||
| } | ||
| }, |
There was a problem hiding this comment.
Review min_provider_count for EIGEN market
Similarly, the EIGEN market is configured with min_provider_count: 1, which could lead to price reliability issues.
Consider increasing the min_provider_count to 3:
"ticker": {
"currency_pair": {
"Base": "EIGEN",
"Quote": "USD"
},
"decimals": 9,
"enabled": true,
"metadata_JSON": "{\"reference_price\":3648941500,\"liquidity\":3099304,\"aggregate_ids\":[{\"venue\":\"coinmarketcap\",\"ID\":\"30494\"}]}",
- "min_provider_count": 1
+ "min_provider_count": 3
}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "EIGEN/USD": { | |
| "provider_configs": [ | |
| { | |
| "invert": false, | |
| "name": "okx_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "EIGEN-USDT" | |
| }, | |
| { | |
| "invert": false, | |
| "name": "bybit_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "EIGENUSDT" | |
| }, | |
| { | |
| "name": "crypto_dot_com_ws", | |
| "off_chain_ticker": "EIGEN_USD" | |
| }, | |
| { | |
| "name": "coinbase_ws", | |
| "off_chain_ticker": "EIGEN-USD" | |
| }, | |
| { | |
| "name": "kraken_api", | |
| "off_chain_ticker": "EIGENUSD" | |
| } | |
| ], | |
| "ticker": { | |
| "currency_pair": { | |
| "Base": "EIGEN", | |
| "Quote": "USD" | |
| }, | |
| "decimals": 9, | |
| "enabled": true, | |
| "metadata_JSON": "{\"reference_price\":3648941500,\"liquidity\":3099304,\"aggregate_ids\":[{\"venue\":\"coinmarketcap\",\"ID\":\"30494\"}]}", | |
| "min_provider_count": 1 | |
| } | |
| }, | |
| "EIGEN/USD": { | |
| "provider_configs": [ | |
| { | |
| "invert": false, | |
| "name": "okx_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "EIGEN-USDT" | |
| }, | |
| { | |
| "invert": false, | |
| "name": "bybit_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "EIGENUSDT" | |
| }, | |
| { | |
| "name": "crypto_dot_com_ws", | |
| "off_chain_ticker": "EIGEN_USD" | |
| }, | |
| { | |
| "name": "coinbase_ws", | |
| "off_chain_ticker": "EIGEN-USD" | |
| }, | |
| { | |
| "name": "kraken_api", | |
| "off_chain_ticker": "EIGENUSD" | |
| } | |
| ], | |
| "ticker": { | |
| "currency_pair": { | |
| "Base": "EIGEN", | |
| "Quote": "USD" | |
| }, | |
| "decimals": 9, | |
| "enabled": true, | |
| "metadata_JSON": "{\"reference_price\":3648941500,\"liquidity\":3099304,\"aggregate_ids\":[{\"venue\":\"coinmarketcap\",\"ID\":\"30494\"}]}", | |
| "min_provider_count": 3 | |
| } | |
| }, |
| }, | ||
| { | ||
| "params": { | ||
| "atomic_resolution": -6, | ||
| "default_funding_ppm": 0, | ||
| "id": 300, | ||
| "liquidity_tier": 4, | ||
| "market_id": 300, | ||
| "market_type": 2, | ||
| "ticker": "EIGEN-USD" | ||
| } | ||
| }, | ||
| { | ||
| "params": { | ||
| "atomic_resolution": -3, | ||
| "default_funding_ppm": 0, | ||
| "id": 301, | ||
| "liquidity_tier": 4, | ||
| "market_id": 301, | ||
| "market_type": 2, | ||
| "ticker": "BOME-USD" | ||
| } |
There was a problem hiding this comment.
💡 Codebase verification
Review atomic resolution and funding rate configurations
Based on the verification results:
-
The atomic resolution of -3 for BOME-USD is indeed an outlier. Most markets use values between -5 and -9, with -6 being the most common (49 instances), followed by -5 (32 instances).
-
Regarding default_funding_ppm, while 100 is common (33 instances), having 0 is actually not unusual (104 instances). This suggests that zero funding rate is a valid configuration choice and doesn't require adjustment.
Recommendation:
- Consider adjusting BOME-USD's atomic_resolution from -3 to a value between -5 and -6 to align with similar markets and ensure appropriate price precision.
🔗 Analysis chain
Review atomic resolution and funding rate configurations
Two potential concerns in the perpetual configurations:
- BOME's atomic_resolution (-3) is unusually high compared to other markets (typically -6 to -10)
- Both markets have default_funding_ppm set to 0, while other markets use 100
Consider adjusting these parameters:
"params": {
"atomic_resolution": -3,
- "default_funding_ppm": 0,
+ "default_funding_ppm": 100,
"id": 301,
"liquidity_tier": 4,
"market_id": 301,
"market_type": 2,
"ticker": "BOME-USD"
}Let me verify the default funding rate configurations before making the final response.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check the range of atomic_resolution values across all markets
rg -U "atomic_resolution.*-\d+" --json | jq -r '.data.lines.text' | sort | uniq -c
Length of output: 5615
Script:
#!/bin/bash
# Check the range of default_funding_ppm values across all markets
rg -U "default_funding_ppm.*\d+" --json | jq -r '.data.lines.text' | sort | uniq -c
Length of output: 4738
| "BOME/USD": { | ||
| "provider_configs": [ | ||
| { | ||
| "invert": false, | ||
| "metadata_JSON": "", | ||
| "name": "kucoin_ws", | ||
| "normalize_by_pair": { | ||
| "Base": "USDT", | ||
| "Quote": "USD" | ||
| }, | ||
| "off_chain_ticker": "BOME-USDT" | ||
| }, | ||
| { | ||
| "invert": false, | ||
| "metadata_JSON": "", | ||
| "name": "huobi_ws", | ||
| "normalize_by_pair": { | ||
| "Base": "USDT", | ||
| "Quote": "USD" | ||
| }, | ||
| "off_chain_ticker": "bomeusdt" | ||
| }, | ||
| { | ||
| "invert": false, | ||
| "metadata_JSON": "", | ||
| "name": "bybit_ws", | ||
| "normalize_by_pair": { | ||
| "Base": "USDT", | ||
| "Quote": "USD" | ||
| }, | ||
| "off_chain_ticker": "BOMEUSDT" | ||
| }, | ||
| { | ||
| "invert": false, | ||
| "metadata_JSON": "{\"base_token_vault\":{\"token_vault_address\":\"FBba2XsQVhkoQDMfbNLVmo7dsvssdT39BMzVc2eFfE21\",\"token_decimals\":6},\"quote_token_vault\":{\"token_vault_address\":\"GuXKCb9ibwSeRSdSYqaCL3dcxBZ7jJcj6Y7rDwzmUBu9\",\"token_decimals\":9},\"amm_info_address\":\"DSUvc5qf5LJHHV5e2tD184ixotSnCnwj7i4jJa4Xsrmt\",\"open_orders_address\":\"38p42yoKFWgxw2LCbB96wAKa2LwAxiBArY3fc3eA9yWv\"}", | ||
| "name": "raydium_api", | ||
| "normalize_by_pair": { | ||
| "Base": "SOL", | ||
| "Quote": "USD" | ||
| }, | ||
| "off_chain_ticker": "BOME,RAYDIUM,UKHH6C7MMYIWCF1B9PNWE25TSPKDDT3H5PQZGZ74J82/SOL,RAYDIUM,SO11111111111111111111111111111111111111112" | ||
| }, | ||
| { | ||
| "invert": false, | ||
| "metadata_JSON": "", | ||
| "name": "okx_ws", | ||
| "normalize_by_pair": { | ||
| "Base": "USDT", | ||
| "Quote": "USD" | ||
| }, | ||
| "off_chain_ticker": "BOME-USDT" | ||
| } | ||
| ], | ||
| "ticker": { | ||
| "currency_pair": { | ||
| "Base": "BOME", | ||
| "Quote": "USD" | ||
| }, | ||
| "decimals": 12, | ||
| "enabled": true, | ||
| "metadata_JSON": "{\"reference_price\":6051284618,\"liquidity\":748591,\"aggregate_ids\":[{\"venue\":\"coinmarketcap\",\"ID\":\"29870\"}]}", | ||
| "min_provider_count": 1 | ||
| } | ||
| }, |
There was a problem hiding this comment.
Review min_provider_count for BOME market
The BOME market is configured with min_provider_count: 1, which is lower than the standard min_provider_count: 3 used by other markets. This could potentially impact price reliability.
Consider increasing the min_provider_count to 3 for better price accuracy and manipulation resistance:
"ticker": {
"currency_pair": {
"Base": "BOME",
"Quote": "USD"
},
"decimals": 12,
"enabled": true,
"metadata_JSON": "{\"reference_price\":6051284618,\"liquidity\":748591,\"aggregate_ids\":[{\"venue\":\"coinmarketcap\",\"ID\":\"29870\"}]}",
- "min_provider_count": 1
+ "min_provider_count": 3
}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "BOME/USD": { | |
| "provider_configs": [ | |
| { | |
| "invert": false, | |
| "metadata_JSON": "", | |
| "name": "kucoin_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "BOME-USDT" | |
| }, | |
| { | |
| "invert": false, | |
| "metadata_JSON": "", | |
| "name": "huobi_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "bomeusdt" | |
| }, | |
| { | |
| "invert": false, | |
| "metadata_JSON": "", | |
| "name": "bybit_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "BOMEUSDT" | |
| }, | |
| { | |
| "invert": false, | |
| "metadata_JSON": "{\"base_token_vault\":{\"token_vault_address\":\"FBba2XsQVhkoQDMfbNLVmo7dsvssdT39BMzVc2eFfE21\",\"token_decimals\":6},\"quote_token_vault\":{\"token_vault_address\":\"GuXKCb9ibwSeRSdSYqaCL3dcxBZ7jJcj6Y7rDwzmUBu9\",\"token_decimals\":9},\"amm_info_address\":\"DSUvc5qf5LJHHV5e2tD184ixotSnCnwj7i4jJa4Xsrmt\",\"open_orders_address\":\"38p42yoKFWgxw2LCbB96wAKa2LwAxiBArY3fc3eA9yWv\"}", | |
| "name": "raydium_api", | |
| "normalize_by_pair": { | |
| "Base": "SOL", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "BOME,RAYDIUM,UKHH6C7MMYIWCF1B9PNWE25TSPKDDT3H5PQZGZ74J82/SOL,RAYDIUM,SO11111111111111111111111111111111111111112" | |
| }, | |
| { | |
| "invert": false, | |
| "metadata_JSON": "", | |
| "name": "okx_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "BOME-USDT" | |
| } | |
| ], | |
| "ticker": { | |
| "currency_pair": { | |
| "Base": "BOME", | |
| "Quote": "USD" | |
| }, | |
| "decimals": 12, | |
| "enabled": true, | |
| "metadata_JSON": "{\"reference_price\":6051284618,\"liquidity\":748591,\"aggregate_ids\":[{\"venue\":\"coinmarketcap\",\"ID\":\"29870\"}]}", | |
| "min_provider_count": 1 | |
| } | |
| }, | |
| "BOME/USD": { | |
| "provider_configs": [ | |
| { | |
| "invert": false, | |
| "metadata_JSON": "", | |
| "name": "kucoin_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "BOME-USDT" | |
| }, | |
| { | |
| "invert": false, | |
| "metadata_JSON": "", | |
| "name": "huobi_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "bomeusdt" | |
| }, | |
| { | |
| "invert": false, | |
| "metadata_JSON": "", | |
| "name": "bybit_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "BOMEUSDT" | |
| }, | |
| { | |
| "invert": false, | |
| "metadata_JSON": "{\"base_token_vault\":{\"token_vault_address\":\"FBba2XsQVhkoQDMfbNLVmo7dsvssdT39BMzVc2eFfE21\",\"token_decimals\":6},\"quote_token_vault\":{\"token_vault_address\":\"GuXKCb9ibwSeRSdSYqaCL3dcxBZ7jJcj6Y7rDwzmUBu9\",\"token_decimals\":9},\"amm_info_address\":\"DSUvc5qf5LJHHV5e2tD184ixotSnCnwj7i4jJa4Xsrmt\",\"open_orders_address\":\"38p42yoKFWgxw2LCbB96wAKa2LwAxiBArY3fc3eA9yWv\"}", | |
| "name": "raydium_api", | |
| "normalize_by_pair": { | |
| "Base": "SOL", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "BOME,RAYDIUM,UKHH6C7MMYIWCF1B9PNWE25TSPKDDT3H5PQZGZ74J82/SOL,RAYDIUM,SO11111111111111111111111111111111111111112" | |
| }, | |
| { | |
| "invert": false, | |
| "metadata_JSON": "", | |
| "name": "okx_ws", | |
| "normalize_by_pair": { | |
| "Base": "USDT", | |
| "Quote": "USD" | |
| }, | |
| "off_chain_ticker": "BOME-USDT" | |
| } | |
| ], | |
| "ticker": { | |
| "currency_pair": { | |
| "Base": "BOME", | |
| "Quote": "USD" | |
| }, | |
| "decimals": 12, | |
| "enabled": true, | |
| "metadata_JSON": "{\"reference_price\":6051284618,\"liquidity\":748591,\"aggregate_ids\":[{\"venue\":\"coinmarketcap\",\"ID\":\"29870\"}]}", | |
| "min_provider_count": 3 | |
| } | |
| }, |
|
@Mergifyio backport release/protocol/v8.x |
✅ Backports have been created
|
(cherry picked from commit 7117942)
|
@Mergifyio backport release/protocol/v7.x |
✅ Backports have been created
|
(cherry picked from commit 7117942) # Conflicts: # protocol/scripts/genesis/sample_pregenesis.json
…port #2651) (#2656) Co-authored-by: Teddy Ding <[email protected]>
…port #2651) (#2658) Co-authored-by: Teddy Ding <[email protected]>
Changelist
Copied over EIGEN,BOME params from prod
Test Plan
Tested on localnet
Author/Reviewer Checklist
state-breakinglabel.indexer-postgres-breakinglabel.PrepareProposalorProcessProposal, manually add the labelproposal-breaking.feature:[feature-name].backport/[branch-name].refactor,chore,bug.Summary by CodeRabbit
New Features
Bug Fixes
ClobPairandPerpetualentries, simplifying validation processes.Documentation
Tests
ClobPairandPerpetualIDs, focusing on other validation aspects.