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

[NA] API key error handling #78

Merged
merged 4 commits into from
Oct 31, 2023
Merged

Conversation

alexkuzmik
Copy link
Collaborator

@alexkuzmik alexkuzmik commented Oct 4, 2023

No description provided.

)

* Make Summary thread safe

* Refactor summary.py, delete logs_registry

* Add incremenet_failed method to Summary

* Refactor failed response handler and introduce exceptions.filter

* Wrap api with errors filter

* Fix lint errors

* Reorganize exception related modules

* Fix lint error

* Add log_message_once=True to api key related exception
@alexkuzmik alexkuzmik self-assigned this Oct 4, 2023
@alexkuzmik alexkuzmik changed the title [CM-8405] Comet LLM log prompt behavior when the api key is missing (… API key error handling Oct 4, 2023
@alexkuzmik alexkuzmik changed the title API key error handling [NA] API key error handling Oct 4, 2023
@alexkuzmik alexkuzmik merged commit 696bf57 into main Oct 31, 2023
13 checks passed
@andrescrz andrescrz deleted the feature/api-key-error-handling branch September 2, 2024 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant