search

gekkowarez / gekkoga

Genetic Algorithm for solving optimization of trading strategies using Gekko
313 stars 104 forks source link

Use randomExt.pick for candleSize in sample config #15

Closed azizmb closed 6 years ago

azizmb commented 6 years ago

Minor cleanup, since randomExt already exists, might as well use it to get the candleSize.

generalectric commented 6 years ago

Very nice! I apologize for the late response.. If you don't mind fixing the conflict I'll get it merged in

azizmb commented 6 years ago

@generalectric Resolved, but was too lazy to do it on my machine. Will cleanup the history and push tomorrow.

generalectric commented 6 years ago

Ha! OK sounds great and do you mind rebasing to develop branch?

generalectric commented 6 years ago

tried it out doesnt work in my case as i have more than 1 candlesize setting and end up with the same number for all my candlesizes... BUT, it does work for normal use with only 1 candlesize.

edit which really confused me because it SHOULD work and led to me trying again and works fine..... yeah its bedtime!