AI4Finance-Foundation / AI4Finance-Foundation/FinRL-Tutorials

future leakage?

Open
#51 1 comment 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
Jupyter Notebook
Stars
1.2k
Forks
435
PR merge metrics
No merged PRs in 30d

Description

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

Contributor guide

Open the contributing guide

Research direction

Locate the NeurIPS2018 demo and trace how close, low, and high values become states. Check their timestamps against the decision and reward periods to determine whether future data is used. Done means the leakage is confirmed or ruled out with the affected code and timing clearly documented.

Written by the indexing model from the issue text.

Assessment

Tech stack
jupyter-notebook
Domain
data, machine-learning
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.