AI4Finance-Foundation / AI4Finance-Foundation/FinRL-Meta

[Suggestion] Include timestamp / DatetimeIndex for plotting

オープン
#48 コメント 10 件 リアクション 2 件 担当者 1 名 @zhumingpassional が担当を希望しています GitHub で見る
good first issue suggestion
主要言語
Python
スター
1.9k
フォーク
751
PR マージ指標
30日以内にマージされた PR はありません

説明

After some more exploring, one thing I realized is, that plotting (with pyfolio / quantstats) to evaluate the results is harder than it needs to be. Currently we have no timestamps easily available which are needed for direct plotting or using the above libraries.

The Demo_FinRL_Meta_Integrate_Trends_data_to_DOW_Jones.ipynb is a good example.
It is done there, but with a (hacky) workaround using a Custom_DataProcessor / the yahoofinance dataframe timestamps.

A solution I could think of would be adding 3 functions to the env that return equal weight portfolio returns, buy-and-hold returns and the agent-returns including the timestamps. A pd.Series with DateTimeindex could work.

This would also be a good preparation for more advanced reward functions like (differential / deflated) sharpe, comparing to buy-and-hold etc. that rely on a return series.

What do you think?

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。