robaho / go-trader

financial exchange written in Go, designed for algorithmic trading tests
GNU General Public License v3.0
376 stars 75 forks source link

Unable to compile due to gocui version #13

Closed napoleon665 closed 2 months ago

napoleon665 commented 2 months ago

Hi @robaho ,

Im see that you have your own custom fork however when I call go install I get the following errors:

image
napoleon665 commented 2 months ago

@robaho Im confused because it should work as your fork has the missing methods - such as maxlines

https://github.com/robaho/gocui/blob/24bdf44ec8485b466af27750a479458c0dc4ab95/view.go#L76

robaho commented 2 months ago

yea, I am confused as well. I am looking into it.

robaho commented 2 months ago

It is fixed now. I needed to make some changes to the module versions. Thanks for reporting, I guess this stopped compiling when I added module support to go-trader

napoleon665 commented 2 months ago

Thanks perfect @robaho , I actually fixed these locally was wondering if u want help maintaining this ?

robaho commented 2 months ago

Thanks perfect @robaho , I actually fixed these locally was wondering if u want help maintaining this ?

I am always up for PRs. I have thought about a few enhancements, like adding a live exchange connection (e.g. Coinbase), and a basic portfolio manager, to allow easier live and backtest strategy testing.