Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Определение цены сделки #1

Open
ffeast opened this issue Feb 14, 2019 · 7 comments
Open

Определение цены сделки #1

ffeast opened this issue Feb 14, 2019 · 7 comments
Labels
enhancement New feature or request

Comments

@ffeast
Copy link
Owner

ffeast commented Feb 14, 2019

Необходимо как-то вытаскивать цену исполнения ордера.
Кроме того, может быть несколько цен исполнения, надо уметь вытаскивать их все
Помимо полного списка цен исполнения и объемов можно выдавать средневзвешенную цену исполнения

@spvik
Copy link

spvik commented Feb 18, 2019

Автор данного фреймоврка выпустил вторую версию, которая имеет много изменений, где многие вопросы уже решены, советую посмотреть https://github.com/BetterQuik/framework

@ffeast
Copy link
Owner Author

ffeast commented Feb 19, 2019

Я видел, часть вещей я перенесу оттуда - как минимум, багфиксы и MarketData через подписку, а ряд изменений я не понял - например, зачем он убрал Indicator, какое отношение к либе имеет json-парсер (если надо - его можно отдельно подключать). При этом SmartOrder без изменений.

Вообщем эта версия исходно проще для понимая идей автора и покрытия автотестами + для моих нужд хватает, поэтому я от нее отпачковался

@spvik
Copy link

spvik commented Feb 20, 2019

У SmartOrder вашей версии есть одна проблема, если исполнения ордера на бирже происходит в одно время с исполнением, не всегда приходит результат о выставление ордера и тем самым теряется id сделки, и SmartOrder списывает прошлую попытку на ошибку и выставляет новую заявку, но в новом фреймворке это исправлено, и в моем форке проверено на реале все четко работает, можно развивать старую версию, но смарт ордер и все коллбеки нужно либо перенести с новой либо исправлять ,на горьком опыте все пройдено!!! )))

@ffeast
Copy link
Owner Author

ffeast commented Feb 21, 2019

Интересный кейс. Это вот для этого в новой версии order определяется до вызова транзакции?
вот тут https://github.com/BetterQuik/framework/blob/master/src/engine.lua#L534
а было https://github.com/spvik/hacktrade/blob/develop/hacktrade.lua#L293

@ffeast
Copy link
Owner Author

ffeast commented Feb 24, 2019

Вообщем перенес этот fix и закрыл тестом
https://github.com/ffeast/hacktrade/blob/develop/tests.lua#L84

@spvik
Copy link

spvik commented Feb 25, 2019

Еще посмотри коллбак OnOrder там есть расчет позиции и определение цены открытой сделки, и в SmartOrder есть методы для получение цены открытой сделки SmartOrder:price_balance(market_price), тоже добавь к себе

@ffeast
Copy link
Owner Author

ffeast commented Mar 7, 2019

Видел это, заберу - пока более важные задачи есть

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants