cog icon indicating copy to clipboard operation
cog copied to clipboard

Report the reason for the Internal Server Error, if an Exception is thrown by predict()

Open cmdr2 opened this issue 3 years ago • 0 comments

Hi,

This is a minor change for reporting the reason for an Internal Server Error, if an Exception is thrown by predict(). This is really helpful for REST API users to understand what failed, especially when using models created by others.

make test-python passed successfully.

Also tested this works correctly with a local docker container running the Stable Diffusion model from replicate, by triggering a scenario where it's supposed to throw an Exception.

I tested by modifying the docker container with this PR, created a docker image and container from that, and tested that it no longer threw an opaque "Internal Server Error" message, but instead described the error correctly.

Thanks

Signed-off-by: cmdr2 [email protected]

cmdr2 avatar Aug 25 '22 18:08 cmdr2