Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More EpochMetric's compute_fn output types #1757

Open
vfdev-5 opened this issue Mar 8, 2021 · 3 comments
Open

More EpochMetric's compute_fn output types #1757

vfdev-5 opened this issue Mar 8, 2021 · 3 comments

Comments

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Mar 8, 2021

🚀 Feature

Following #1700 (comment) we would like to improve EpochMetric by supporting more output types returned by compute_fn:

  • EpochMetric should work with compute_fn outputs: scalars, tensors, tuple/list/mapping of tensors vs currently only scalars

A code snippet on possible implementation: #1700 (comment) (It is not tested and maybe wrong, just an idea of how to do it).

It would be also helpful to raise a more user friendly error message if output type is not supported.

We need to update the docs as well.

If interested in fixing this issue, please leave a comment to assign the issue to you.

@sdesrozis
Copy link
Contributor

@vfdev-5 I can help on this if needed.

@vfdev-5
Copy link
Collaborator Author

vfdev-5 commented Mar 8, 2021

Thanks @sdesrozis , I mentioned this issue in #1729 and asked @Moh-Yakoub if he'd like to give a try...

@vfdev-5
Copy link
Collaborator Author

vfdev-5 commented Oct 9, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants