Skip to content

Conversation

@Zid-0
Copy link
Contributor

@Zid-0 Zid-0 commented Oct 18, 2019

Backtesting in cli didn't work. I fixed this.toml variable initialization and some variable typos.
Now backtesting in cli works as expected.

Zid-0 added 2 commits October 18, 2019 13:46
Fixed this.toml initialisation and corrected some variables. Now backtesting in cli works.
@PGTBoos
Copy link
Owner

PGTBoos commented Oct 18, 2019

I've not yet ran it in cli (will soon when my rasbPi arrives i a few weeks),
However the TOML variables are used a lot in the code.
For example look at the lines around 286 and later.
If i'm right about the current usage of JS this refers to the main object of the strategy.
By doing this.toml i can use variables at all locations (they're global).
I think the code would brake with the changes sugested. (missing out on lines where those variables are used)

@RadiantAeon
Copy link
Contributor

Noticed this issue too. Will test once my mini hdmi cable arrives for my raspberry pi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants