Skip to content

Commit

Permalink
Setting mktdata=NULL in walk.forward.R
Browse files Browse the repository at this point in the history
apply.paramset and applyStrategy will now apply the appropriate mktdata related to each symbol in the portfolio, as opposed to just the first.

See #125
  • Loading branch information
jaymon0703 committed Aug 25, 2020
1 parent 11f7ebf commit 226e45e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions R/walk.forward.R
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ walk.forward <- function( strategy.st
, paramset.label=paramset.label
, portfolio.st=portfolio.st
, account.st=account.st
, mktdata=symbol.data
, mktdata=NULL
, rule.subset=training.timespan
, nsamples=nsamples
, calc='slave'
Expand Down Expand Up @@ -323,7 +323,7 @@ walk.forward <- function( strategy.st
applyStrategy( strategy
, portfolios=test.portfolio.st
# , portfolios=portfolio.st
, mktdata=symbol.data
, mktdata=NULL
, rule.subset=testing.timespan
, ...
)
Expand Down Expand Up @@ -380,7 +380,7 @@ walk.forward <- function( strategy.st
, paramset.label=paramset.label
, portfolio.st=portfolio.st
, account.st=account.st
, mktdata=symbol.data
, mktdata=NULL
, nsamples=nsamples
, calc='slave'
, audit=.insampleaudit
Expand Down

0 comments on commit 226e45e

Please sign in to comment.