fix: use equity field for balance in Unified Account API #6
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.
Fix balance handling for Bybit Unified Account API
Issue
The current implementation uses
availableToWithdrawfield from the Bybit API response to get account balance. However, with the Unified Account API, this field is often returned as an empty string, causing the code to fail when trying to convert it to float.Fix
Modified
get_asset_balancemethod to use theequityfield instead ofavailableToWithdraw. Theequityfield is consistently populated with the correct balance value in the Unified Account API response.Changes
get_asset_balancemethod inbybit_store.pyto useequityinstead ofavailableToWithdrawTesting
Tested with Bybit Unified Account using both spot and linear perpetual markets. The fix allows the code to properly retrieve account balances and proceed with trading operations.