odp icon indicating copy to clipboard operation
odp copied to clipboard

[PATCH v2] example: classifier: improvements for performance testing

Open JereLeppanen opened this issue 2 years ago • 2 comments

Some improvements to the classifier example, mainly to enable performance testing.

JereLeppanen avatar Oct 07 '22 08:10 JereLeppanen

Perhaps a more stringent CLI input validation could be considered if this is now becoming a proper tester

No more proper than it already was. I don't think we should add a bunch of input validation code unless we really need it for some reason.

JereLeppanen avatar Oct 14 '22 08:10 JereLeppanen

About the CLI error checking, for users, it is better to have sane error messages logged by the tester logic instead of something weird or undefined behavior from implementations. Doesn't have to be this PR though.

TuomasTaipale avatar Oct 14 '22 11:10 TuomasTaipale