Ignite: Remove useless prints from tests

Created on 17 Aug 2020  路  3Comments  路  Source: pytorch/ignite

馃殌 Feature

Some tests contain prints that probably were used for debugging and were not removed. It is better to remove them.

For example,
https://github.com/pytorch/ignite/blob/44c60564bf59a3028ab89091cbd3d73def8be86f/tests/ignite/contrib/engines/test_common.py#L71

https://github.com/pytorch/ignite/blob/44c60564bf59a3028ab89091cbd3d73def8be86f/tests/ignite/contrib/engines/test_common.py#L244

https://github.com/pytorch/ignite/blob/44c60564bf59a3028ab89091cbd3d73def8be86f/tests/ignite/contrib/handlers/test_param_scheduler.py#L631

etc

Hacktoberfest good first issue help wanted
Source
picture vfdev-5

Most helpful comment

@vfdev-5 I am working on this.

picture Yogendra0Sharma on 17 Aug 2020
👍2

All 3 comments

@vfdev-5 I am working on this.

Yogendra0Sharma picture Yogendra0Sharma on 17 Aug 2020
👍2

Thanks @Yogendra0Sharma ! Feel free to send a draft PR to iterate over if you wish

vfdev-5 picture vfdev-5 on 17 Aug 2020

This issue is still open as previous draft PR could not be finalized by the author.

vfdev-5 picture vfdev-5 on 8 Sep 2020
Was this page helpful?
0 / 5 - 0 ratings

Related issues

samarth-robo picture samarth-robo  路  3Comments
czotti picture czotti  路  3Comments
TheCodez picture TheCodez  路  3Comments
kilsenp picture kilsenp  路  3Comments
UjwalKandi picture UjwalKandi  路  3Comments