AI4Finance-Foundation / AI4Finance-Foundation/FinRL-Meta
Delete obsolete duplicates in FinRL vs FinRL-Meta
- Lenguaje dominante
- Python
- Estrellas
- 1.9k
- Forks
- 751
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
1)
I cannot understand why there is FinRL and FinRL-Meta. Why you don't merge them? Duplicates:
https://github.com/AI4Finance-Foundation/FinRL/blob/master/tutorials/3-Practical/FinRL_MultiCrypto_Trading.ipynb
https://github.com/AI4Finance-Foundation/FinRL-Meta/blob/master/tutorials_notebook/3-Practical/FinRL_MultiCrypto_Trading.ipynb
https://github.com/AI4Finance-Foundation/FinRL-Meta/blob/master/tutorials_python/3-Practical/FinRL_MultiCrypto_Trading.py seems empty
Tutorial FinRL:
Seems it doesn't use FinRL. Than why it's in FinRL repo, if it use only FinRL-Meta?
DataProcessor in FinRL doesn't support Binance.
For what is better to use FinRL and for what FinRL-Meta?
FinRL is obsolete (vs FinRL-Meta) and I can just ignore it and use FinRL-Meta only?
Why do you support 2 similar repos with similar structure, but one is more updated than another?
By your info in README.md https://github.com/AI4Finance-Foundation/FinRL I think that I need FinRL, but seems FinRL-Meta is actual and better version of FinRL. FinRL obsolete (vs FinRL-Meta)
2)
CryptoEnv there is in tutorial, but also it's available from finrl_meta.env_crypto_trading.env_multiple_crypto import CryptoEnv . Which one is more updated and better to use?
https://github.com/AI4Finance-Foundation/FinRL/blob/master/finrl/finrl_meta/env_cryptocurrency_trading/env_multiple_crypto.py
https://github.com/AI4Finance-Foundation/FinRL-Meta/blob/master/finrl_meta/env_crypto_trading/env_multiple_crypto.py
Why doesn't delete it from tutorial and just import from actual repo?
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Evaluación
Este issue todavía no se ha evaluado.