Skip to content

Use logger instead of print in experiments - #60

Merged
muupan merged 3 commits into
chainer:masterfrom
muupan:use-logger-in-experiments
Mar 28, 2017
Merged

Use logger instead of print in experiments#60
muupan merged 3 commits into
chainer:masterfrom
muupan:use-logger-in-experiments

Conversation

@muupan

@muupan muupan commented Mar 28, 2017

Copy link
Copy Markdown
Member

Resolves #32

  • Replace print sentences in chainerrl.experiments module with logger.info so that you can configure it.
  • Modify the quickstart notebook to print info messages into stdout as before.
  • Additionally, use gym.undo_logger_setup in other examples for consistency and clearness.

@coveralls

coveralls commented Mar 28, 2017

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.08%) to 57.146% when pulling da41b6b on muupan:use-logger-in-experiments into bd9631a on pfnet:master.

@muupan
muupan merged commit 011d2c5 into chainer:master Mar 28, 2017
@muupan
muupan deleted the use-logger-in-experiments branch March 28, 2017 13:24
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