Fixed take profit calculation. Added a position check for the current pair when trying to change the margin.
This commit is contained in:
@@ -296,10 +296,10 @@ async def open_positions(
|
|||||||
if (liq_long > 0 or liq_short > 0) and price_for_cals > 0:
|
if (liq_long > 0 or liq_short > 0) and price_for_cals > 0:
|
||||||
if side == "Buy":
|
if side == "Buy":
|
||||||
base_tp = price_for_cals + (price_for_cals - liq_long)
|
base_tp = price_for_cals + (price_for_cals - liq_long)
|
||||||
take_profit_price = base_tp + commission_fee_percent
|
take_profit_price = base_tp + commission_fee_percent / qty_formatted
|
||||||
else:
|
else:
|
||||||
base_tp = price_for_cals - (liq_short - price_for_cals)
|
base_tp = price_for_cals - (liq_short - price_for_cals)
|
||||||
take_profit_price = base_tp - commission_fee_percent
|
take_profit_price = base_tp - commission_fee_percent / qty_formatted
|
||||||
take_profit_price = max(take_profit_price, 0)
|
take_profit_price = max(take_profit_price, 0)
|
||||||
else:
|
else:
|
||||||
take_profit_price = None
|
take_profit_price = None
|
||||||
@@ -307,10 +307,10 @@ async def open_positions(
|
|||||||
stop_loss_price = None
|
stop_loss_price = None
|
||||||
else:
|
else:
|
||||||
if side == "Buy":
|
if side == "Buy":
|
||||||
take_profit_price = price_for_cals * (1 + take_profit_percent / 100) + commission_fee_percent
|
take_profit_price = price_for_cals * (1 + take_profit_percent / 100) + commission_fee_percent / qty_formatted
|
||||||
stop_loss_price = price_for_cals * (1 - stop_loss_percent / 100)
|
stop_loss_price = price_for_cals * (1 - stop_loss_percent / 100)
|
||||||
else:
|
else:
|
||||||
take_profit_price = price_for_cals * (1 - take_profit_percent / 100) - commission_fee_percent
|
take_profit_price = price_for_cals * (1 - take_profit_percent / 100) - commission_fee_percent / qty_formatted
|
||||||
stop_loss_price = price_for_cals * (1 + stop_loss_percent / 100)
|
stop_loss_price = price_for_cals * (1 + stop_loss_percent / 100)
|
||||||
|
|
||||||
take_profit_price = max(take_profit_price, 0)
|
take_profit_price = max(take_profit_price, 0)
|
||||||
|
@@ -41,11 +41,26 @@ class TelegramMessageHandler:
|
|||||||
if order_status == "Filled" or order_status not in status_map:
|
if order_status == "Filled" or order_status not in status_map:
|
||||||
return None
|
return None
|
||||||
|
|
||||||
|
user_auto_trading = await rq.get_user_auto_trading(
|
||||||
|
tg_id=tg_id, symbol=symbol
|
||||||
|
)
|
||||||
|
auto_trading = (
|
||||||
|
user_auto_trading.auto_trading if user_auto_trading else False
|
||||||
|
)
|
||||||
|
user_deals_data = await rq.get_user_deal_by_symbol(
|
||||||
|
tg_id=tg_id, symbol=symbol
|
||||||
|
)
|
||||||
|
|
||||||
text = (
|
text = (
|
||||||
f"Торговая пара: {symbol}\n"
|
f"Торговая пара: {symbol}\n"
|
||||||
f"Количество: {qty}\n"
|
|
||||||
f"Движение: {side_rus}\n"
|
f"Движение: {side_rus}\n"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
if user_deals_data is not None and auto_trading:
|
||||||
|
text += f"Текущая ставка: {user_deals_data.order_quantity}\n"
|
||||||
|
else:
|
||||||
|
text += f"Количество: {qty}\n"
|
||||||
|
|
||||||
if price and price != "0":
|
if price and price != "0":
|
||||||
text += f"Цена: {price}\n"
|
text += f"Цена: {price}\n"
|
||||||
if take_profit and take_profit != "Нет данных":
|
if take_profit and take_profit != "Нет данных":
|
||||||
@@ -185,7 +200,7 @@ class TelegramMessageHandler:
|
|||||||
"Risk is too high for this trade": "❗️ Риск сделки слишком высок для продолжения",
|
"Risk is too high for this trade": "❗️ Риск сделки слишком высок для продолжения",
|
||||||
"ab not enough for new order": "❗️ Недостаточно средств для продолжения торговли",
|
"ab not enough for new order": "❗️ Недостаточно средств для продолжения торговли",
|
||||||
"InvalidRequestError": "❗️ Недостаточно средств для размещения нового ордера с заданным количеством и плечом.",
|
"InvalidRequestError": "❗️ Недостаточно средств для размещения нового ордера с заданным количеством и плечом.",
|
||||||
"The number of contracts exceeds maximum limit allowed": "❗️ Количество контрактов превышает допустимое максимальное количество контрактов",
|
"The number of contracts exceeds maximum limit allowed": "❗️ Превышен максимальный лимит ставки",
|
||||||
}
|
}
|
||||||
error_text = errors.get(
|
error_text = errors.get(
|
||||||
res, "❗️ Не удалось открыть новую сделку"
|
res, "❗️ Не удалось открыть новую сделку"
|
||||||
|
@@ -7,6 +7,7 @@ from aiogram.types import CallbackQuery, Message
|
|||||||
import app.telegram.keyboards.inline as kbi
|
import app.telegram.keyboards.inline as kbi
|
||||||
import database.request as rq
|
import database.request as rq
|
||||||
from app.bybit.get_functions.get_instruments_info import get_instruments_info
|
from app.bybit.get_functions.get_instruments_info import get_instruments_info
|
||||||
|
from app.bybit.get_functions.get_positions import get_active_positions_by_symbol, get_active_orders_by_symbol
|
||||||
from app.bybit.set_functions.set_leverage import set_leverage
|
from app.bybit.set_functions.set_leverage import set_leverage
|
||||||
from app.bybit.set_functions.set_margin_mode import set_margin_mode
|
from app.bybit.set_functions.set_margin_mode import set_margin_mode
|
||||||
from app.helper_functions import is_int, is_number, safe_float
|
from app.helper_functions import is_int, is_number, safe_float
|
||||||
@@ -211,6 +212,31 @@ async def settings_for_margin_type(
|
|||||||
"""
|
"""
|
||||||
try:
|
try:
|
||||||
await state.clear()
|
await state.clear()
|
||||||
|
symbol = await rq.get_user_symbol(tg_id=callback_query.from_user.id)
|
||||||
|
deals = await get_active_positions_by_symbol(
|
||||||
|
tg_id=callback_query.from_user.id, symbol=symbol
|
||||||
|
)
|
||||||
|
position = next((d for d in deals if d.get("symbol") == symbol), None)
|
||||||
|
|
||||||
|
if position:
|
||||||
|
size = position.get("size", 0)
|
||||||
|
else:
|
||||||
|
size = 0
|
||||||
|
|
||||||
|
if safe_float(size) > 0:
|
||||||
|
await callback_query.answer(
|
||||||
|
text="У вас есть активная позиция по текущей паре",
|
||||||
|
)
|
||||||
|
return
|
||||||
|
|
||||||
|
orders = await get_active_orders_by_symbol(
|
||||||
|
tg_id=callback_query.from_user.id, symbol=symbol)
|
||||||
|
|
||||||
|
if orders is not None:
|
||||||
|
await callback_query.answer(
|
||||||
|
text="У вас есть активный ордер по текущей паре",
|
||||||
|
)
|
||||||
|
return
|
||||||
await callback_query.message.edit_text(
|
await callback_query.message.edit_text(
|
||||||
text="Выберите тип маржи:\n\n"
|
text="Выберите тип маржи:\n\n"
|
||||||
"Примечание: Если у вас есть открытые позиции, то маржа примениться ко всем позициям",
|
"Примечание: Если у вас есть открытые позиции, то маржа примениться ко всем позициям",
|
||||||
|
@@ -47,7 +47,7 @@ async def start_trading(callback_query: CallbackQuery, state: FSMContext) -> Non
|
|||||||
|
|
||||||
if safe_float(size) > 0:
|
if safe_float(size) > 0:
|
||||||
await callback_query.answer(
|
await callback_query.answer(
|
||||||
text="У вас есть активная позиция",
|
text="У вас есть активная позиция по текущей паре",
|
||||||
)
|
)
|
||||||
return
|
return
|
||||||
|
|
||||||
@@ -56,7 +56,7 @@ async def start_trading(callback_query: CallbackQuery, state: FSMContext) -> Non
|
|||||||
|
|
||||||
if orders is not None:
|
if orders is not None:
|
||||||
await callback_query.answer(
|
await callback_query.answer(
|
||||||
text="У вас есть активные ордера",
|
text="У вас есть активный ордер по текущей паре",
|
||||||
)
|
)
|
||||||
return
|
return
|
||||||
|
|
||||||
@@ -94,13 +94,14 @@ async def start_trading(callback_query: CallbackQuery, state: FSMContext) -> Non
|
|||||||
)
|
)
|
||||||
|
|
||||||
error_messages = {
|
error_messages = {
|
||||||
"Limit price is out min price": "Цена лимитного ордера меньше минимального",
|
"Limit price is out min price": "Цена лимитного ордера меньше допустимого",
|
||||||
"Limit price is out max price": "Цена лимитного ордера больше максимального",
|
"Limit price is out max price": "Цена лимитного ордера больше допустимого",
|
||||||
"Risk is too high for this trade": "Риск сделки превышает допустимый убыток",
|
"Risk is too high for this trade": "Риск сделки превышает допустимый убыток",
|
||||||
"estimated will trigger liq": "Лимитный ордер может вызвать мгновенную ликвидацию. Проверьте параметры ордера.",
|
"estimated will trigger liq": "Лимитный ордер может вызвать мгновенную ликвидацию. Проверьте параметры ордера.",
|
||||||
"ab not enough for new order": "Недостаточно средств для создания нового ордера",
|
"ab not enough for new order": "Недостаточно средств для создания нового ордера",
|
||||||
"InvalidRequestError": "Произошла ошибка при запуске торговли.",
|
"InvalidRequestError": "Произошла ошибка при запуске торговли.",
|
||||||
"Order does not meet minimum order value": "Сумма ставки меньше допустимого для запуска торговли.",
|
"Order does not meet minimum order value": "Сумма ставки меньше допустимого для запуска торговли. "
|
||||||
|
"Увеличьте ставку, чтобы запустить торговлю",
|
||||||
"position idx not match position mode": "Измените режим позиции, чтобы запустить торговлю",
|
"position idx not match position mode": "Измените режим позиции, чтобы запустить торговлю",
|
||||||
"Qty invalid": "Некорректное значение ставки для данного инструмента",
|
"Qty invalid": "Некорректное значение ставки для данного инструмента",
|
||||||
"The number of contracts exceeds maximum limit allowed": "️️Превышен максимальный лимит ставки",
|
"The number of contracts exceeds maximum limit allowed": "️️Превышен максимальный лимит ставки",
|
||||||
|
Reference in New Issue
Block a user