search

notadamking / RLTrader

A cryptocurrency trading environment using deep reinforcement learning and OpenAI's gym
https://discord.gg/ZZ7BGWh
GNU General Public License v3.0
1.73k stars 540 forks source link

why was _reset_session removed from the environment? #55

Closed maxmatical closed 5 years ago

maxmatical commented 5 years ago

In the earlier version of the environment, there was a

def _reset_session(self)

that allowed the environment to start at a random point in the dataframe to get more training samples. why was this removed for the newest version of the btc environment?

notadamking commented 5 years ago

This was removed when the idea of a "trading session" was removed. It could be a nice experiment to see the results of adding it back in.