wardbradt / peregrine

Detects arbitrage opportunities across 131 cryptocurrency exchanges in 50 countries
MIT License
1.18k stars 339 forks source link

Setting starting currency no longer works. #49

Closed LPX55 closed 5 years ago

LPX55 commented 5 years ago

Been a while since I played with this project. One issue I'm running into, which wasn't a problem before, is that setting the starting currency for one exchange, multiple coins seems to no longer work properly.

Whether its

paths = bellman_ford(graph, 'BTC', unique_paths=True) or paths = bellman_ford(graph, 'ETH', unique_paths=True)

or any currency I put, it seems that the starting currency is different everytime depending on the current opportunities.

wardbradt commented 5 years ago

Hi, Bellman Ford finds opportunities which are reachable from a currency. It does not find opportunities containing a currency. If an opportunity is found containing a currency you would like to start from, it can be easily be done by rotating the list of currencies.

wardbradt commented 5 years ago

Can I close this?

parthi2929 commented 5 years ago

Hi

If so, what is the purpose of that currency input?

LPX55 commented 5 years ago

Hi

If so, what is the purpose of that currency input?

Wondering the same thing...