-
Notifications
You must be signed in to change notification settings - Fork 261
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Is there a roadmap for using cryptocurrency trading data? #25
Comments
@djoffrey,
Expanded:
|
Hello, @Kismuz I changed the code in order to be able to use daily data frames instead of intradaily data frames. I can see that the episode.data contains a correct sample of my daily dataframe and this one is accepted however it remains in global step 0 and always geting new random samples. |
@joaosalvado10 - any details? what script you are running? shots of terminal and tensor_board output? Upd: I looked at the file. It may be irrelevant to this particular issue, but it should be properly sorted, while this one is not: records 1-29 are for 1970's and from 30 it goes 1800'th up. |
I think I solved the problem, I am trying to change the state given to the agent by including features available in the csv file. |
Hi Kismuz, I have found https://github.com/bartosh/backtrader/tree/ccxt which has support for a wide range of crypto exchanges. I would be interested to know if btgym supports the same exchanges as backtrader. Thank you for your work! |
@kazi308, |
@Kismuz function read_csv => function load_data would be nicer, I think. |
Can you share details of what data you use and mods have been done?
You mean adding other than csv source of input or more changing parsing approach to existing data source? |
My dataset is a pandas.DataFrame object with datetime index, and OHLCV columns, I manually replace "data" of BTGymDataSet and related member vars etc.
Yes, I think a general data pipeline is quite important. |
Hi , @joaosalvado10 ,could you please give a simple example to show how to use daily data with btgym? I cannot figure it out yet. Thanks! |
No description provided.
The text was updated successfully, but these errors were encountered: