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

chore: log openai completion error #3476

Merged
merged 1 commit into from
Nov 27, 2024
Merged

Conversation

zwpaper
Copy link
Member

@zwpaper zwpaper commented Nov 27, 2024

we can know what is the error when completion failed

2024-11-26T17:46:53.617182Z  WARN http_api_bindings::completion::openai: crates/http-api-bindings/src/completion/openai.rs:107: Error in completion event source: 404 Not Found, {  "error": {    "message": "This is a chat model and not supported in the v1/completions endpoint. Did you mean to use v1/chat/completions?",    "type": "invalid_request_error",    "param": "model",    "code": null  }}

@wsxiaoys wsxiaoys enabled auto-merge (squash) November 27, 2024 02:38
@wsxiaoys wsxiaoys merged commit e5f3e06 into main Nov 27, 2024
5 of 6 checks passed
@wsxiaoys wsxiaoys deleted the chore/log-openai-completion-error branch November 27, 2024 02:45
Copy link

codecov bot commented Nov 27, 2024

Codecov Report

Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Project coverage is 62.95%. Comparing base (b6b8d60) to head (20dabf5).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
crates/http-api-bindings/src/completion/openai.rs 0.00% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3476      +/-   ##
==========================================
- Coverage   62.96%   62.95%   -0.01%     
==========================================
  Files         194      194              
  Lines       22182    22188       +6     
==========================================
+ Hits        13966    13968       +2     
- Misses       8216     8220       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

2 participants