Skip to content

Commit

Permalink
comment update
Browse files Browse the repository at this point in the history
  • Loading branch information
vzhomeexperiments committed Dec 16, 2020
1 parent a511300 commit c784066
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
## Planned Changes

* Add folder generation function
* Put option predict ahead as a parameter (now 34 bars)
* aml_make_model use parameter to split data with specific n of rows e.g. n_rows

# lazytrade 0.4.3

Expand All @@ -12,6 +14,7 @@

* solving warnings by changing example in *mt_make_model* function
* change function *mt_make_model* to read manually checked dataset
* update documentation

# lazytrade 0.4.2

Expand Down
2 changes: 1 addition & 1 deletion R/aml_make_model.R
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ aml_make_model <- function(symbol, timeframe, path_model, path_data,


# split data to train and test blocks
# note: model will be tested on the PAST data and trained on the NEWEST data
# note: model will be trained on the OLDEST data
test_ind <- 1:round(0.3*(nrow(dat12))) #train indices 1:xxx
dat21 <- dat12[test_ind, ] #dataset to test the model using 30% of data
dat22 <- dat12[-test_ind, ] #dataset to train the model
Expand Down

0 comments on commit c784066

Please sign in to comment.