AI4Finance-Foundation / FinRL-Meta

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

Update test.py #231

Closed mhdmyz closed 1 year ago

mhdmyz commented 1 year ago

I highly suggest for simplifying and easier debugging and better readability and understandability of the codes, the environment creation gets it's own function and gets separated from the train and test functions. One of the prominent advantages of FinRL-Meta is it's modularity and having different independent layers, which the proposed change helps going further in this direction.

YangletLiu commented 1 year ago

@mhdmyz thanks very much! We will do a code review soon!