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

Change model prediction output from tuple to dict #84

Open
streino opened this issue Jul 5, 2022 · 0 comments
Open

Change model prediction output from tuple to dict #84

streino opened this issue Jul 5, 2022 · 0 comments

Comments

@streino
Copy link
Contributor

streino commented Jul 5, 2022

Change originally proposed in #73

Current model outputs a (scores, labels) prediction tuple.

I suggest changing it to a {'labels': labels, 'scores': scores} dict. This makes output more explicit and follows what we've done training-side in the dataset format and model inputs.

It will change the model output in serving as well, so it would require some matching changes in the code that uses the model for prediction.

@streino streino changed the title Change model prediction output from (scores, labels) to {'labels': labels, 'scores': scores} Change model prediction output from tuple to dict Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant