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 539 forks source link

Changed the indicators to be included through a list. #105

Closed mwbrulhardt closed 5 years ago

mwbrulhardt commented 5 years ago

Hi Adam,

I thought it might be useful to include the indicators by keeping them as a list of tuples where first is the name of the indicator, then the function that calculates them, and then the arguments they need holding them as a list of tuples. It might be helpful in the pursuit of making the features more pluggable. Let me know what you think.

notadamking commented 5 years ago

I like it :)