google / dopamine

Dopamine is a research framework for fast prototyping of reinforcement learning algorithms.
https://github.com/google/dopamine
Apache License 2.0
10.42k stars 1.36k forks source link

DEPRECATION WARNING: Logger is being deprecated. Please switch to CollectorDispatcher! #198

Closed FelipeMartins96 closed 2 years ago

FelipeMartins96 commented 2 years ago

I wanted to report this deprecation warning as I got it when running the training agents example commands from the docs, I'm not sure about how would I proceed to switch to CollectorDispatcher, not sure if there is a implementation available currently to use as reference.

psc-g commented 2 years ago

hi, we are replacing the previous Logger https://github.com/google/dopamine/blob/master/dopamine/discrete_domains/logger.py#L31 with a more flexible setup using CollectorDispatchers https://github.com/google/dopamine/blob/master/dopamine/metrics/collector_dispatcher.py, and this warning is just giving users advance notice. you can safely ignore this warning, but if you'd like to get rid of it you can just set this line to false (or set it with gin_bindings): https://github.com/google/dopamine/blob/master/dopamine/discrete_domains/run_experiment.py#L175

that being said, i agree it's a little cryptic, so we'll look into adding a landing page that explains the transition.

On Tue, May 24, 2022 at 11:07 AM Felipe Martins @.***> wrote:

I wanted to report this deprecation warning as I got it when running the training agents example commands from the docs, I'm not sure about how would I proceed to switch to CollectorDispatcher, not sure if there is a implementation available currently to use as reference.

— Reply to this email directly, view it on GitHub https://github.com/google/dopamine/issues/198, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE3CCMJPAZY6WSUDJS4QB5TVLTWEHANCNFSM5WZ5C3LQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

FelipeMartins96 commented 2 years ago

got it! thanks. I will look into de docstring on collector_dispatcher.py to understand it better. So the agents of this repo are already using the CollectorDispatchers right?

psc-g commented 2 years ago

yup, they're already using CollectorsDispatcher, we just kept the legacy logger activated for now, to avoid breaking people :)

On Wed, May 25, 2022 at 2:04 PM Felipe Martins @.***> wrote:

got it! thanks. I will look into de docstring on collector_dispatcher.py to understand it better. So the agents of this repo are already using the CollectorDispatchers right?

— Reply to this email directly, view it on GitHub https://github.com/google/dopamine/issues/198#issuecomment-1137655848, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE3CCMNYPIB7K3NGDHOECQTVLZTRRANCNFSM5WZ5C3LQ . You are receiving this because you commented.Message ID: @.***>

FelipeMartins96 commented 2 years ago

Thank you for the info, @psc-g; I will close this issue then, as it is not an issue, in fact :smile: