Ignite: Improve docs for common.setup_common_training_handlers

Created on 29 Aug 2020  路  10Comments  路  Source: pytorch/ignite

馃殌 Feature

Idea is to fix rendering (see below) and add an example of usage and a link where it is used.

Screen Shot 2020-08-29 at 22 56 07

What is wrong: gray frame with red vertical line should be absent for the text "Helper method to setup trainer with common handlers "

https://pytorch.org/ignite/contrib/engines.html#ignite.contrib.engines.common.setup_common_training_handlers

Same for other methods:

  • common.setup_common_distrib_training_handlers
  • common.setup_*_logging

For Hacktoberfest contributors, feel free to ask questions for details if any and say that you would like to tackle the issue.
Please, take a look at CONTRIBUTING guide.

Hacktoberfest enhancement good first issue help wanted

All 10 comments

Assign it to me I shall work on it.@vfdev-5

@rexdivakar sure

@rexdivakar any updates on this ?

Working on this will raise a PR soon

@rexdivakar sounds good. It would be nice to have the PR on tuesday. As you have 3 issues assigned we'd like to have some activity on these issues without blocking other users who'd like to help out.

oh k will do it soon @vfdev-5

@rexdivakar could you please send a PR for this issue within 2 days, otherwise unfortunately I'll have to unassign you.

I鈥檇 like to look into this

@vfdev-5 should the gray frame with red vertical line be present for "Parameters" title?

@BanzaiTokyo yes, this is OK, see the docs for other correctly rendered methods, e.g. https://pytorch.org/ignite/contrib/engines.html#ignite.contrib.engines.tbptt.create_supervised_tbptt_trainer

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Sudy picture Sudy  路  4Comments

elanmart picture elanmart  路  4Comments

vfdev-5 picture vfdev-5  路  3Comments

andreydung picture andreydung  路  4Comments

samarth-robo picture samarth-robo  路  3Comments