Fix default logger

Question

Fix default logger

Closed this issue 6 years ago · 4 comments

Current default is essentially /dev/null which is probably not the expected behavior for people trying to run reaver from inside their own codebase.

Answer 1 · 2019-01-12T04:29:59.000Z

So is this the cause of the AttributeError: 'AdvantageActorCriticAgent' object has no attribute 'logger' when trying to run a quickstart currently?

Answer 2 · 2019-01-12T11:05:21.000Z

Oh, seems I've broken it with some of the recent changes. As a temporary workaround you can manually initialize the /dev/null logger via agent.logger = rvr.utils.Logger(). Alternatively see how the full StreamLogger is initialized here.

Answer 3 · 2019-01-12T18:07:14.000Z

Thank you so much!

Answer 4 · 2019-01-15T15:12:18.000Z

Enabled StreamLogger by default with sensible configs in 2.1.5.