Skip to content
This repository has been archived by the owner on Feb 16, 2020. It is now read-only.

Updated binance market data using script provided in PR #1550 by @cmroche #1700

Merged
merged 2 commits into from
Jan 17, 2018
Merged

Updated binance market data using script provided in PR #1550 by @cmroche #1700

merged 2 commits into from
Jan 17, 2018

Conversation

mronus
Copy link
Contributor

@mronus mronus commented Jan 16, 2018

Fix for #1678

Updated binance market data using the script provided in PR #1550 by @cmroche

This was referenced Jan 16, 2018
@askmike
Copy link
Owner

askmike commented Jan 17, 2018

👍 thanks :)

@askmike askmike merged commit 8396106 into askmike:develop Jan 17, 2018
@mronus mronus deleted the update/binance_market_data branch January 17, 2018 18:56
@askmike askmike mentioned this pull request Jan 21, 2018
@nissimpradhan
Copy link

Can binance market data using the script be done automatically when using import option form the Gekko UI? I think that would be great. If it hasn't been done, l'd love to do it myself.

@askmike
Copy link
Owner

askmike commented Feb 7, 2018

No, you have to manually run the update file.

The reason we do it like this is so that we can make sure Gekko only works with markets that we have tested and seen. We could also dynamically look up markets whenever you start Gekko, but that defeats the purpose.

@nissimpradhan
Copy link

What do you mean by tested and seen? Binance keeps adding new coins to the exchange, Gekko should support those trading pairs isn't it? I thought that was the purpose of the script.

It's not a ideal to run it every time we use the importer but perhaps, a "sync currency pairs" button in the import page definitely helps.

@askmike
Copy link
Owner

askmike commented Feb 7, 2018 via email

@askmike
Copy link
Owner

askmike commented Feb 7, 2018 via email

@jbghub
Copy link

jbghub commented Feb 15, 2018

I am new to Gekko. This discussion is very confusing. I would like to see more coins available in Gekko for trading listed by Bianance. If an update has been prepared that will accomplish this I would like to know how to install the update. Thanks

@stereohelix
Copy link
Contributor

jbghub, here's a way to update for more coins without having to use any other tools:

-Make a backup copy of gekko/exchanges/binance.js
-Then open your current file, copy the contents of this into it, and save:
https://github.com/askmike/gekko/blob/c3de6f653930b1a8a58ef5071cfd0537664dc7d3/exchanges/binance.js

@jbghub
Copy link

jbghub commented Feb 28, 2018 via email

@jbghub
Copy link

jbghub commented Feb 28, 2018

I updated the binance.js file as stated. However, I am still only seeing the same 5 coins in the Gekko app. Any suggestions?

@stereohelix
Copy link
Contributor

Stop and restart gekko?

@jbghub
Copy link

jbghub commented Feb 28, 2018

Yes, Stopped, Restarted Gekko; also rebooted PC. Any other ideas?

@jbghub
Copy link

jbghub commented Feb 28, 2018

I am using the Admin Command window to start Gekko:
node gekko --ui
then going to http://localhost:3000/#/home in Chrome browzer.

@stereohelix
Copy link
Contributor

Hi jbghub,
Sorry, now I'm confused about this. I don't know how you could only show five coins after restarting, since the binance.js file is the only file where that information is read from. This might be beyond my ability to hack. Are you sure you edited the correct file? The only other thing I could suggest is restoring the files, pulling gecko again fresh (using git pull) and then trying again.

Good luck, hope that helps.

@jbghub
Copy link

jbghub commented Mar 1, 2018

I found this code in the package-lock file:
"binance": { "version": "1.3.1", "resolved": "https://registry.npmjs.org/binance/-/binance-1.3.1.tgz"

Could this have anything to do with the binance file not working for me?

@stereohelix
Copy link
Contributor

I'm not sure. But that is a little out-of-date.

You could try
npm uninstall binance
npm install binance

But I should really add a disclaimor that I'm in over my head here -- if this causes some other conflict, you might end up having to reinstall gekko. Which might be fine, and might actually solve whatever issue is happening.

@jbghub
Copy link

jbghub commented Mar 1, 2018

I believe I found my problem. I was only looking at the USDT pairs. Their are many more coins for other currencies (coins) instead of USDT.

@jbghub
Copy link

jbghub commented Mar 1, 2018

Thanks for all your help.

@stereohelix
Copy link
Contributor

Great!

@jbghub
Copy link

jbghub commented Mar 1, 2018

Quesion: How did you determine the Precision of each coin?

@stereohelix
Copy link
Contributor

The easiest way is to look on the Binance website. The precision is the least significance digit of the price.

@cmroche
Copy link
Contributor

cmroche commented Mar 1, 2018

@stereohelix @jbghub Right now it happens to be the least significant digit of each coin, but Binance does publish somewhere in their documents and official list, and it is queryable through their API for official values as there is no guarantee of this relationship in the future.

@stereohelix
Copy link
Contributor

Here's the official document (it's actually pretty up-to-date, not 8 months old)
https://support.binance.com/hc/en-us/articles/115000594711-Trading-Rule

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants