Skip to content

Fixes #1010 and #1022 #1034

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 15, 2023
Merged

Fixes #1010 and #1022 #1034

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c505d42
After [this](https://github.com/DLR-RM/stable-baselines3/commit/9c338…
AkashKarnatak Jun 10, 2023
311b1f4
DummyVecEnv.render() no longer returns the expected state after [this…
AkashKarnatak Jun 10, 2023
4c8d9b6
Merge branch 'master' of github.com:AI4Finance-Foundation/FinRL
AkashKarnatak Jun 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Merge branch 'master' of github.com:AI4Finance-Foundation/FinRL
  • Loading branch information
@AkashKarnatak
AkashKarnatak committed Jun 15, 2023
commit 4c8d9b62771c0c498ae5bbd9d02e29a9dfc5e718
7 changes: 6 additions & 1 deletion finrl/meta/env_stock_trading/env_stocktrading.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -355,7 +355,12 @@ def step(self, actions):

return self.state, self.reward, self.terminal, False, {}

def reset(self, seed=None):
def reset(
self,
*,
seed=None,
options=None,
):
# initiate state
self.state = self._initiate_state()

Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.