Tinkoff / invest-python

Tinkoff Invest Python gRPC client
https://tinkoff.github.io/invest-python/
Apache License 2.0
319 stars 90 forks source link

docs(examples): Live strategy for several tickers. Strategy prints OHLCV for every completed candle. #205

Closed WISEPLAT closed 1 year ago

WISEPLAT commented 1 year ago

Привет всем! Хотелось бы добавить в примеры:

Пример live стратегии для нескольких тикеров. Вывод OHLCV для каждой сформировавшейся свечи.

Прошу рассмотреть и подтвердить. Спасибо!

WISEPLAT commented 1 year ago

Всё пофиксил )) прошу заапрувить)

Привет всем! Хотелось бы добавить в примеры:

Пример live стратегии для нескольких тикеров. Вывод OHLCV для каждой сформировавшейся свечи.

Прошу рассмотреть и подтвердить. Спасибо!

WISEPLAT commented 1 year ago

CI падает, надо разобраться почему и пофиксить

Поправил. Прошу заапрувить.

WISEPLAT commented 1 year ago

предлагаю это заапрувить - потом сделать версию 2.0 - улучшенную))) с доп оптимизацией.

irusland commented 1 year ago

предлагаю это заапрувить - потом сделать версию 2.0 - улучшенную))) с доп оптимизацией.

Предлагаю сразу доводить до ума, чтобы тех долга не копилось.

Комменты сверху все же надо порешать.

WISEPLAT commented 1 year ago

предлагаю это заапрувить - потом сделать версию 2.0 - улучшенную))) с доп оптимизацией.

Предлагаю сразу доводить до ума, чтобы тех долга не копилось.

Комменты сверху все же надо порешать.

как это вернуть?

return AsyncServices( channel, token=self._token, sandbox_token=self._sandbox_token, app_name=self._app_name, )

WISEPLAT commented 1 year ago

какие принципиальные не решены? Чтобы понимать. - а то здесь уже путаешься в кол-ве ...

irusland commented 1 year ago

какие принципиальные не решены? Чтобы понимать. - а то здесь уже путаешься в кол-ве ...

Те, которые не пофикшены

WISEPLAT commented 1 year ago

так я все исправил

irusland commented 1 year ago

Давайте хотя бы добьём AsyncClient

WISEPLAT commented 1 year ago

Давайте хотя бы добьём AsyncClient

можно, какое предложение? если нет возвращаемого объекта: return AsyncServices( channel, token=self._token, sandbox_token=self._sandbox_token, app_name=self._app_name, )

от async def aenter(self) -> AsyncServices:

WISEPLAT commented 1 year ago

раз по другому не получается вернуть такой объект - посмотрел в исходниках, то может пример всё же заапрувим - ведь это позволит народу сэкономить кучу времени на создание прототипа структуры своей стратегии используя API Тинькофф

WISEPLAT commented 1 year ago

Давайте хотя бы добьём AsyncClient

Апрувим? ) Надеюсь на +

WISEPLAT commented 1 year ago

Апрувим?

Давайте хотя бы добьём AsyncClient

Добрый день! Т.к. по другому не получается вернуть такой объект - посмотрел в исходниках, то может пример всё же заапрувим?

irusland commented 1 year ago

по другому не получается вернуть такой объект

какой объект вам необходимо вернуть?

WISEPLAT commented 1 year ago

нет возвращаемого объекта: return AsyncServices( channel, token=self._token, sandbox_token=self._sandbox_token, app_name=self._app_name, )

от async def aenter(self) -> AsyncServices:

нет возвращаемого объекта: return AsyncServices( channel, token=self._token, sandbox_token=self._sandbox_token, app_name=self._app_name, )

от async def aenter(self) -> AsyncServices:

поэтому используется метод

.aenter()

для строки:

client = await AsyncClient(token=TOKEN, app_name="TinkoffApp").aenter()

я думаю такое исправление не принципиально для примера, и предлагаю уже заапрувить.

irusland commented 1 year ago

К сожалению я не смог понять что значит нет возвращаемого объекта поэтому подготовил фикс который просил вас сделать. Примените его fix(review)__use_propper_async_with.patch

WISEPLAT commented 1 year ago

К сожалению я не смог понять что значит нет возвращаемого объекта поэтому подготовил фикс который просил вас сделать. Примените его fix(review)__use_propper_async_with.patch

Готово. Протестил - всё работает.

irusland commented 1 year ago

Поддержите мастер пожалуйста, чтобы тесты прошли.

WISEPLAT commented 1 year ago

Поддержите мастер пожалуйста, чтобы тесты прошли.

Добрый день! Готово.

WISEPLAT commented 1 year ago

Can you please merge this pull request? P.S. It is approved by @irusland