AI4Finance-Foundation / AI4Finance-Foundation/FinRL-Tutorials

FinRL_MultiCrypto_Trading.py no longer working due to changes in ElegantRL and FinRL-Meta

Abierto
#61 1 comentario 3 reacciones 0 asignados Ver en GitHub
bug
Lenguaje dominante
Jupyter Notebook
Estrellas
1.2k
Forks
435
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Hi, there is quite a lot broken in the current version of FinRL_MultiCrypto_Trading.ipynb

See: https://github.com/AI4Finance-Foundation/FinRL-Meta/issues/291
Also see: #21

I am able to go around many of the problems by running old versions of the repos/dependencies but in the end i failed.
As mentioned in one of the urls. Since Jan 2023 ElegantRL is no longer compatible.

/FinRL-Meta/agents/elegantrl_models.py is also no longer compatible due to line 58 (and more) changing Arguments(agent=agent, env=env) to Arguments(agent_class=agent, env=env)

ray.rllib also seems to have had some breaking changes. Also tried version 2.0.0 since its from around date the tutorial was moved into this repo but still didn't work.

Please fix the tutorials. I realy would like to get started looking into FinRL.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Start with FinRL_MultiCrypto_Trading.ipynb and compare its dependencies with /FinRL-Meta/agents/elegantrl_models.py, especially the Arguments call described in the issue. Review linked issues #291 and #21, then run the tutorial to identify the remaining ElegantRL and ray.rllib incompatibilities; done means the tutorial completes with current dependencies.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
jupyter-notebook
Área
fintech-quant, machine-learning
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.