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

This version of the bot is now broken, pending merge of the localDB branch #1

Closed
kuzetsa opened this issue Dec 31, 2013 · 6 comments
Closed
Assignees

Comments

@kuzetsa
Copy link
Owner

kuzetsa commented Dec 31, 2013

see also:

askmike#90

This forked repo now much more closely follows the askmike/gekko branch:

d3ef7e8

Progress is now stalled until localDB branch is usable. full stop.

leaving issue #1 open until at least 1 branch on this fork is usable again.

@ghost ghost assigned kuzetsa Dec 31, 2013
@kuzetsa
Copy link
Owner Author

kuzetsa commented Dec 31, 2013

In case it's not clear, this is what I'm waiting on:

https://github.com/askmike/gekko/tree/localDB

LOTS of changes, because multiple exchanges have made drastic changes to how things work, and more than one exchange is now 100% incompatible on an API level with the older code used in earlier versions of gekko bot.

kuzetsa referenced this issue in askmike/gekko Dec 31, 2013
@kuzetsa
Copy link
Owner Author

kuzetsa commented Dec 31, 2013

askmike@919af4d#commitcomment-4963515

I'm not sure how to even re-add support for live trading.

EVERYTHING is a total rewrite on the localDB branch from what I can tell.

@kuzetsa
Copy link
Owner Author

kuzetsa commented Feb 21, 2014

LocalDB initial merge seems to be in mainline now.

Submitted request for comments: askmike#212

@kuzetsa
Copy link
Owner Author

kuzetsa commented Jun 25, 2014

Working on the pull for the final "localDB-based" merge RIGHT NOW!!!

Moving forward, not going to support any exchanges other than cex.io, and behavior during the start of a downtrend will be unspecified (no longer any advice or trading to "sell high"... or any "sell" under any circumstance. GHS reinvestment will be the only strategy, though it should theoretically work with any trading method)

@kuzetsa
Copy link
Owner Author

kuzetsa commented Jun 25, 2014

Initial fork 6b8e2fc completed, bugfixes to follow.

@kuzetsa kuzetsa closed this as completed Jun 25, 2014
@kuzetsa
Copy link
Owner Author

kuzetsa commented Jun 25, 2014

The tag "initial.fork.2014.june" is the current bugfix release I'm using / testing as of 2014 June 25th.

Seems to have fixed all the major bugs (as well as cosmetic ones relating to display, and more consistent behavior down to the API-specified 8 decimal places worth of precision, etc.)

kuzetsa referenced this issue Jun 25, 2014
1) Added several lines of code related to bugs with inconsistent
formatting / display / logging of information, and made the verbose
"debug" logging less... just less (much of information is now in
the default level without needing to enabled debug logging)... Also
changed the grammar on various things to prevent linewrapping
from too much words, or simple cleanups to make 8 decimal places be
usd correctly (in addition to fixing a few rounding errors)

2) Hardcoded a trading fee for cex.io platform, as it now has one (this
value doesn't seem to have a way to fetch via API, so I guess it can be
configurable in the future. The config parser needs an audit though, so
the workaround was to simply hardcode a non-zero fee)

3) Removed the "sell when price is high" logic. This fork of gekkobot is
for reinvestment of the income from cloud mining. Period. Full stop.
kuzetsa pushed a commit that referenced this issue Aug 26, 2016
Merging latest commits from source (askmike/gekko)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant