search

AI4Finance-Foundation / FinRL-Tutorials

Tutorials. Please star.
https://ai4finance.org
MIT License
848 stars 349 forks source link

future leakage? #51

Open charliedream1 opened 1 year ago

charliedream1 commented 1 year ago

In demo NeurIPS2018, close/low/high value are used as states, it seems future leakage exist?

YangletLiu commented 1 year ago

This issue is discussed at FinRL: https://github.com/AI4Finance-Foundation/FinRL/issues/938