AI4Finance-Foundation / FinRL-Meta

FinRL­-Meta: Dynamic datasets and market environments for FinRL.
https://ai4finance.org
MIT License
1.28k stars 583 forks source link

fix for binance not included in the assert condition and return imports #38

Closed ma7555 closed 2 years ago

ma7555 commented 2 years ago

This fixes uncompleted listed of supported exchanges, ex: 'binance' from the assert condition. It should be removed as the same check is already handled in: https://github.com/AI4Finance-Foundation/FinRL-Meta/blob/90eb1289313f07488f2e905ae071c64441124536/finrl_meta/data_processor.py#L14

Also, re-add the technical indicators imports as they are used. I suggest integrating a PR process as some PRs introduce problems.

Regards, ma7555