这是indexloc提供的服务,不要输入任何密码
Skip to content

add examples, fix some bugs in the last PR #5

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.

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 21 commits into from
Mar 27, 2020

Conversation

Mehooz
Copy link
Contributor

@Mehooz Mehooz commented Mar 27, 2020

  • I have marked all applicable categories:
    • exception-raising fix
    • algorithm implementation fix
    • documentation modification
    • new feature
  • If applicable, I have mentioned the relevant/related issue(s)

Less important but also useful:

  • I have visited the source website, and in particular read the known issues
  • I have searched through the issue tracker for duplicates
  • I have mentioned version numbers, operating system and environment, where applicable:
    import tianshou, sys
    print(tianshou.__version__, sys.version, sys.platform)

@Trinkle23897 Trinkle23897 merged commit 77068af into thu-ml:master Mar 27, 2020
Comment on lines +37 to +39

batch.returns = (batch.returns - batch.returns.mean()) \
/ (batch.returns.std() + self._eps)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here, your code is the same as the next two lines.

Comment on lines +61 to +63

batch.returns = (batch.returns - batch.returns.mean()) \
/ (batch.returns.std() + self._eps)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above

@huang6668 huang6668 mentioned this pull request May 4, 2023
8 tasks
BFAnas pushed a commit to BFAnas/tianshou that referenced this pull request May 5, 2024
* update atari.py

* fix setup.py
pass the pytest

* fix setup.py
pass the pytest

* add args "render"

* change the tensorboard writter

* change the tensorboard writter

* change device, render, tensorboard log location

* change device, render, tensorboard log location

* remove some wrong local files

* fix some tab mistakes and the envs name in continuous/test_xx.py

* add examples and point robot maze environment

* fix some bugs during testing examples

* add dqn network and fix some args

* change back the tensorboard writter's frequency to ensure ppo and a2c can write things normally

* add a warning to collector

* rm some unrelated files

* reformat

* fix a bug in test_dqn due to the model wrong selection
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