search

yGuy / chatgpt-mattermost-bot

A very simple implementation of a service for a mattermost bot that uses ChatGPT in the backend.
MIT License
145 stars 50 forks source link

Introduce Error Handling for createChatCompletion() thread #40

Open your-highness opened 1 year ago

your-highness commented 1 year ago

Closes #35

your-highness commented 1 year ago

Any idead when it will be merged?

yGuy commented 1 year ago

Can you describe the effect of this PR? How exactly did it improve the situation for you?Before: crash, stacktrace, restart.After: crash, log error, continue. What else?Do you get better error messages this way? -------- Ursprüngliche Nachricht --------Von: Johannes Helmuth @.> Datum: 01.06.23 15:09 (GMT+06:00) An: yGuy/chatgpt-mattermost-bot @.> Cc: Subscribed @.***> Betreff: Re: [yGuy/chatgpt-mattermost-bot] Introduce Error Handling for createChatCompletion() thread (PR #40) Any idead when it will be merged?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

your-highness commented 1 year ago

The log output indicates now that the chat completion did not work properly. Before it was not giving any output at all if case of crashing threads.

yGuy commented 1 year ago

OK. Please update to not just error the status and message, but format all of the available information into a single status and message error. Include a text that describes the location of the error. Like "error from openai response: status: .. message: ..."