1756 update ignite version to 0.4.4

[Nic-Ma]

Fixes #1756 .

Description

A few sentences describing the changes proposed in this pull request.

Status

Ready

Types of changes

• Non-breaking change (fix or new feature that would not break existing functionality).
• Breaking change (fix or new feature that would cause existing functionality to change).
• New tests added to cover the changes.
• Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
• Quick tests passed locally by running ./runtests.sh --quick --unittests.
• In-line docstrings updated.
• Documentation updated, tested make html command in the docs/ folder.

[Nic-Ma]

/black

[Nic-Ma]

/integration-test

[Nic-Ma]

/black

[Nic-Ma]

/integration-test

[Nic-Ma]

Hi @vfdev-5 ,

Thanks for your great work on ignite project.
May I double confirm why you guys changed the default value of arg device from None to CPU in metrics?

Thanks.

[Nic-Ma]

/black

[vfdev-5]

Hi @Nic-Ma , this is a BC-breaking change since 0.4.3 to explicitly manage metric's storage device and reduce the time of moving from CPU to GPU in DDP for reduction etc. Please, see pytorch/ignite#1082

[Nic-Ma]

Hi @vfdev-5 ,

Thanks for your explanation!
Could you please also help review this PR?

Thanks.

[vfdev-5]

Overall PR's impressions is good. I left few questions though (1 is more or less unrelated).

[wyli]

thanks!