ctubio / Krypto-trading-bot

Self-hosted crypto trading bot (automated high frequency market making) written in C++
https://127.0.0.1:3000
Other
3.37k stars 823 forks source link

HitBTC blank wallet #91

Closed Jorman closed 7 years ago

Jorman commented 7 years ago

@ctubio I opened a new issue, the last about sql3 is resolved and locked so here you can see my logs

[thanks for the log!]

ctubio commented 7 years ago

try valgrind node K to see what is going on behind the segfault

Jorman commented 7 years ago

For sure, here the log

K.linux.57
13:54:3.395408 CF settings loaded from K.json OK.
13:54:6.214979 DB /data/db/K.6.0.6.db loaded OK.
13:54:9.17765 UI ready over HTTPS on external port 3000.
==4544== Invalid read of size 8
==4544==    at 0xA3DFB7: v8::Object::Get(v8::Local<v8::Value>) (in /usr/bin/nodejs)
==4544==    by 0x9D805F6: K::GwPoloniex::fetch() (in /home/lello/K/app/server/lib/K.linux.57.node)
==4544==    by 0x9D62BF2: K::GW::main(v8::Local<v8::Object>) (in /home/lello/K/app/server/lib/K.linux.57.node)
==4544== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)
Killed
ctubio commented 7 years ago

ok many thanks, later maaaaybe i will look at K::GwPoloniex::fetch() (im not really interested into invest time in poloniex if you ask me)

ctubio commented 7 years ago

just saying, will rewrite soon the v8 Object of Poloniex to a plain json object, so this may be fixed then, someday

Jorman commented 7 years ago

I tried poloniex because is the only that I've access right now, if you indicate me a good exchange to work with I can try with it. I'm pretty noob with exchange so I don't know if is better to work with Post-Only Orders support or without Post-Only Because I don't really know what is the real difference. What exchange do you suggest?

ctubio commented 7 years ago

bitfinex or okcoin or coinbase/gdax is nice i think

Jorman commented 7 years ago

Just tried with hitbtc here the log

==6709==    by 0x9D89874: void nlohmann::detail::from_json<nlohmann::basic_json<std::map, std::vector, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool, long, unsigned long, double, std::allocator, nlohmann::adl_serializer> >(nlohmann::basic_json<std::map, std::vector, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool, long, unsigned long, double, std::allocator, nlohmann::adl_serializer> const&, nlohmann::basic_json<std::map, std::vector, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool, long, unsigned long, double, std::allocator, nlohmann::adl_serializer>::string_t&) (in /home/lello/K/app/server/lib/K.linux.57.node)
==6709==    by 0x9D89A69: K::CF::cfString(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool) (in /home/lello/K/app/server/lib/K.linux.57.node)
==6709==    by 0x9D643E6: K::QP::main(v8::Local<v8::Object>) (in /home/lello/K/app/server/lib/K.linux.57.node)
==6709== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
Killed
lello@9f60:~/K$

I've some problem with the second ID on coinbase/gdax I'll try on bitfinex too ad I'll report the test

ctubio commented 7 years ago

for me hitbtc works fine

Jorman commented 7 years ago

do you have some idea on what can be?

ctubio commented 7 years ago

no idea, i cant reproduce it; without valgrind doesnt says nothing interesting?

ctubio commented 7 years ago

poloniex should be fixed after https://github.com/ctubio/Krypto-trading-bot/commit/db4dc488e9b9997f63901a0416d07b7595581386

ctubio commented 7 years ago

finally found the hitbtc issue in the IRC channel, fixed at https://github.com/ctubio/Krypto-trading-bot/commit/5c8c08bd0fdcbb6bf68b287615a4a55b96291dc6, well but now the hitbtc wallet is blank

Jorman commented 7 years ago

Wow, you really rock!!! Tried with the latest version, now works both! Now is time to start to understand how it works! Any suggestion?

J

ctubio commented 7 years ago

first suggestion is to read the manual in the root folder :D