A RetroSearch Logo

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Search Query:

Showing content from https://github.com/pytorch/pytorch/issues/99270 below:

Remove lr_scheduler.print_lr · Issue #99270 · pytorch/pytorch · GitHub

📚 The doc issue

The purpose of print_lr is unclear, and arguments are undocumented.
Also, the API is counter-intuitive. For example, in python API, we do not use print('hello', verbose=False) to not print something, and here the verbose is even a required argument. Instead, the user should put an if condition before calling the print function.

Related thread: https://discuss.pytorch.org/t/how-to-use-print-lr-in-the-lr-scheduler/132420
Source: https://github.com/pytorch/pytorch/blob/master/torch/optim/lr_scheduler.py#L113-L124

Suggest a potential alternative/fix

I suggest removing that function since users can still simply get the current LR via get_last_lr and design the printing message on their own.

cc @vincentqb @jbschlosser @albanD @janeyx99


RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4