AI4Finance-Foundation / AI4Finance-Foundation/FinRL-Tutorials

There is a bug in 1-Introduction Stock_NeurIPS2018_SB3.ipynb

Open
#72 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Jupyter Notebook
Stars
1.2k
Forks
435
PR merge metrics
No merged PRs in 30d

Description

![image](https://github.com/AI4Finance-Foundation/FinRL-Tutorials/assets/28076275/e340b06b-fc29-441d-a64f-189e12a8dca7)

MergeError: Passing 'suffixes' which cause duplicate columns {'account_value_x'} is not allowed.

please change to this code:
result = pd.merge(df_result_a2c, df_result_ddpg, left_index=True, right_index=True)
result = pd.merge(result, df_result_td3, left_index=True, right_index=True)
result = pd.merge(result, df_result_ppo, left_index=True, right_index=True, suffixes=('_1', '_2'))
result = pd.merge(result, df_result_sac, left_index=True, right_index=True)
result = pd.merge(result, MVO_result, left_index=True, right_index=True, suffixes=('_3', '_4'))
result = pd.merge(result, df_dji, left_index=True, right_index=True, suffixes=('_5', '_6'))
result.columns = ['a2c', 'ddpg', 'td3', 'ppo', 'sac', 'mean var', 'dji']

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.