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
availableToWithdraw
field 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_balance
method to use theequity
field instead ofavailableToWithdraw
. Theequity
field is consistently populated with the correct balance value in the Unified Account API response.Changes
get_asset_balance
method inbybit_store.py
to useequity
instead ofavailableToWithdraw
Testing
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.