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

Update openai models 20241206 #1968

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

longkeyy
Copy link
Contributor

@longkeyy longkeyy commented Dec 6, 2024

close #1967

image image

Copy link

codecov bot commented Dec 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 1.21%. Comparing base (7c8628b) to head (33af45d).

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1968   +/-   ##
=====================================
  Coverage   1.21%   1.21%           
=====================================
  Files        146     146           
  Lines      12295   12295           
=====================================
  Hits         149     149           
  Misses     12132   12132           
  Partials      14      14           

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

@SimsonW
Copy link

SimsonW commented Dec 12, 2024

只改这些不够。o1 不能获取流式的 token 数,laisky 有写过一个 pr,不知道为什么没有合入

@popdo
Copy link

popdo commented Dec 20, 2024

"o1-preview":              15,
"o1-preview-2024-09-12":   15,
"o1-mini":                 3,
"o1-mini-2024-09-12":      3,

你这倍率全都设错了

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.

同步最新o1-mini,o1-preview模型
3 participants