A-I-P-A / CSRAssist

Tooling to assist CSRs
0 stars 0 forks source link

Add exception handling around ocr call from slackbot #18

Open tramdas opened 10 months ago

tramdas commented 10 months ago

Over on https://github.com/A-I-P-A/CSRAssist/issues/11#issuecomment-1858741501 @sonalikapatel84 described a situation where the ocr integration was not working with her GPU stack:

A initialization: The NVIDIA driver on your system is too old (found version 11040). Please update your GPU driver by downloading and installing a new version from the URL:

We could add some exception handling around https://github.com/A-I-P-A/CSRAssist/blob/main/slack_integration/slack_bot.py#L61 to gracefully handle this or any other OCR failures.

I'd suggest if there is an exception we can just treat the string list as an empty list. While doing this we should also add a check for a non-empty stringlist before calling stringlist.describe_tv here: https://github.com/A-I-P-A/CSRAssist/blob/main/slack_integration/slack_bot.py#L71

We should still print the exception traceback so we can debug any issues arising.