-
Notifications
You must be signed in to change notification settings - Fork 85
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
Update for Gekko 0.6.0 #35
Comments
Indeed it seems to be not working anymore, profit[%] and profit/day[%] are both at 0.00. |
I am getting that error too. @xFFFFF , are you planning on fixing this? |
This is not my tool, maybe @xFFFFF can weigh in?
…On Thu, Jul 12, 2018 at 1:37 PM, jonathanday930 ***@***.***> wrote:
I am getting that error too. @askmike <https://github.com/askmike>, are
you planning on fixing this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#35 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA7MD4rRdZKduY8LCXJfEvXGl1omytRYks5uFuCRgaJpZM4VCYe6>
.
--
PGP key at keybase.io/mikevanrossum
<https://keybase.io/mikevanrossum/key.asc>
|
If I were to use the gekko version right before 0.6, do you think backtesting would still run ok? Im only going to use this installation for backtesting. |
I think so yes, unless you run in any of the issues people have documented
before.
…On Thu, 12 Jul 2018, 13:57 jonathanday930, ***@***.***> wrote:
If I were to use the gekko version right before 0.6, do you think
backtesting would still run ok? Im only going to use this installation for
backtesting.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#35 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA7MD4aZYX_ejdxfiNVJsUQgJh-GKfZYks5uFuU5gaJpZM4VCYe6>
.
|
in lines 1280-1281 of backtest.pl change to works for me |
@xFFFFF looks like a great tool. Thanks.
|
I am also having this issue. I,ve changed the lines 1280-1281 of backtest.pl as suggested by gabrielrey37 above but the results look the same: Profit and Profit/day = 0.00 |
@jonathanday930 did you find a previous Gekko version that works with the latest BackTestTool? Is it sufficient for optimizing strategy parameters? |
@danibeni81 Here's what I use for profit variable that works: |
dear @robmazur thanks a lot!! it seems it works!! |
with 0.65 patch not working |
I removed one of the |
@suprnrdy thanks, it seems ok now. im using Gekko v0.6.6 |
I'm sharing my version of the string which should be updated (1280). I have tested and all works. Old string The new string |
Is there any "ultimate" solution to 0 % profit? I think I've checked every single proposal here yet still 0% profit persists :/ @YuriyTigiev Any other additional change is needed apart from change in 1280 line? |
Maybe you're looking at mine, not a perfect regex, but it works for me (it extracts the percent of profit):
|
@verbraucher Works like charm! Thanks! |
I've released gekko 0.6 which has some differences in the API. See the following links for details:
The text was updated successfully, but these errors were encountered: