Skip to content

Conversation

@AkashKarnatak
Copy link
Contributor

No description provided.

…, DummyVecEnv.reset() method passes a `seed` argument to its list of envs. Since StockTradingEnv.reset() does not expect any argument it results in an error. This commit fixes AI4Finance-Foundation#1022.
@zhumingpassional
Copy link
Collaborator

thanks for your codes.
revison 1) last_state = trade_env.render()
last_state = trade_env.envs[0].render()
you revision aims to render the first env?

@zhumingpassional
Copy link
Collaborator

This branch has conflicts that must be resolved
could you pls revise it?

@AkashKarnatak
Copy link
Contributor Author

AkashKarnatak commented Jun 15, 2023

thanks for your codes. revison 1) last_state = trade_env.render() last_state = trade_env.envs[0].render() you revision aims to render the first env?

yes, it tries to render the first env of DummyVecEnv.

@zhumingpassional zhumingpassional merged commit 5dcac3a into AI4Finance-Foundation:master Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants