LastAncientOne / Deep_Learning_Machine_Learning_Stock

Deep Learning and Machine Learning stocks represent promising opportunities for both long-term and short-term investors and traders.
MIT License
1.26k stars 317 forks source link

:pencil2: Split for X and y #3

Closed Qoyyuum closed 3 years ago

Qoyyuum commented 3 years ago

Found that it was splitting df and y instead of X. Otherwise, X isn't used and the resulting variables wouldn't make sense. Unless its supposed to be df_train and df_test.

LastAncientOne commented 3 years ago

Yes, you are correct. It suppose to be X. I update it.