"test_loss" in test step

[hminle]

Hi PL team,

🚀 Feature

I see in the test step, we should return "test_loss" instead of "val_loss".
It will become clearer.

Thanks

[github-actions]

Hi! thanks for your contribution!, great first issue!

[awaelchli]

could you point us to the link you found this?

[hminle]

could you point us to the link you found this?

Hi @awaelchli ,
Here you are: https://pytorch-lightning.readthedocs.io/en/stable/introduction_guide.html#testing

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[awaelchli]

ok, this will probably anyway be overwritten by the docs in part 3 of #2615 structured results.

[Borda]

@hminle mind send a PR?

[romesco]

This docs fix was overwritten in latest which now recommends returning an EvalResult object: https://pytorch-lightning.readthedocs.io/en/latest/introduction_guide.html#testing

The relevant info is now in:
https://pytorch-lightning.readthedocs.io/en/latest/results.html

I made a PR to clarify that the validation loop paragraph is also applicable to the test_loop.
#2827

[awaelchli]

nice thank you!

[hminle]

Thank guys