machine-learning-for-trading icon indicating copy to clipboard operation
machine-learning-for-trading copied to clipboard

Typos and other Errata

Open casras111 opened this issue 4 years ago • 6 comments

From reading the book below what seem like typos:

In chapter 4 repeated passage image

In chapter 5 calculations of std and variance mixed:

chptr5_2 chprt5_1

casras111 avatar Jan 13 '21 19:01 casras111

Thank you! I'll keep this open in the hope that others will also contribute any errors they come across so I can fix them in the next edition.

stefan-jansen avatar Jan 21 '21 07:01 stefan-jansen

On algoseek_minute_data.ipynb Section 4 Nasdaq 100 constituents, the field date_time is incorrect and should be date_timebarstart errata1

drsxr avatar Jun 01 '21 16:06 drsxr

@drsx the issue was a missing inplace=True when renaming the index to date_time, which I have fixed.

stefan-jansen avatar Jun 07 '21 16:06 stefan-jansen

Not sure if this is a typo or actually intended, but it looks like there are some inconsistencies in the TimeGAN notebook with regards to indexing for the supervisor inputs and outputs:

Screenshot from 2021-10-20 14-41-51

Screenshot from 2021-10-20 14-42-27

Screenshot from 2021-10-20 14-42-40

When training the supervisor, the mse calculation is mse(h[:, 1:, :], h_hat_supervised[:, :-1, :]), but the other two times it is mse(h[:, 1:, :], h_hat_supervised[:, 1:, :]). Is this intentional?

hxyue1 avatar Oct 20 '21 21:10 hxyue1

It looks like in the script: 08_ml4t_workflow/02_vectorized_backtest.ipynb the conditions (">") applied after .rank for selecting the best long_signals and short_signals must be inverted ("<=").

image

dama0418 avatar Feb 08 '22 13:02 dama0418

image

sample perhaps?

CDLim0906 avatar Mar 21 '22 09:03 CDLim0906