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.

Sign up for GitHub

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

Jump to bottom

update cost info #2401

Merged
merged 1 commit into from
Apr 17, 2024
Merged

update cost info #2401

merged 1 commit into from
Apr 17, 2024

Conversation

sonichi
Copy link
Contributor

@sonichi sonichi commented Apr 16, 2024

Why are these changes needed?

Update the cost info for the new gpt-4 model release in April.

Related issue number

Checks

@sonichi sonichi enabled auto-merge April 17, 2024 00:34
@sonichi sonichi requested a review from a team April 17, 2024 00:35
@sonichi sonichi added this pull request to the merge queue Apr 17, 2024
BeibinLi
BeibinLi approved these changes Apr 17, 2024
View reviewed changes
Copy link
Collaborator

@BeibinLi BeibinLi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

I can create a PR for image token count, but that function might need PIL or other image reading packages for the openai_utils. @sonichi Thoughts?

@sonichi
Copy link
Contributor Author

sonichi commented Apr 17, 2024

LGTM.

I can create a PR for image token count, but that function might need PIL or other image reading packages for the openai_utils. @sonichi Thoughts?

Make them optional dependencies :)

Merged via the queue into main with commit 4ab8a88 Apr 17, 2024
63 of 76 checks passed
@sonichi sonichi deleted the cost416 branch April 17, 2024 01:15
jayralencar pushed a commit to jayralencar/autogen that referenced this pull request May 28, 2024
@sonichi
update cost info (microsoft#2401)
991435c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BeibinLi BeibinLi BeibinLi approved these changes

@PyroGenesis PyroGenesis PyroGenesis approved these changes

@ekzhu ekzhu Awaiting requested review from ekzhu

@qingyun-wu qingyun-wu Awaiting requested review from qingyun-wu

@olgavrou olgavrou Awaiting requested review from olgavrou

@yiranwu0 yiranwu0 Awaiting requested review from yiranwu0

@sethupavan12 sethupavan12 Awaiting requested review from sethupavan12

@wmwxwa wmwxwa Awaiting requested review from wmwxwa

@AaronWard AaronWard Awaiting requested review from AaronWard

@RohitRathore1 RohitRathore1 Awaiting requested review from RohitRathore1

@ShobhitVishnoi30 ShobhitVishnoi30 Awaiting requested review from ShobhitVishnoi30

@DustinX DustinX Awaiting requested review from DustinX

@essos-bot essos-bot Awaiting requested review from essos-bot

@Mai0313 Mai0313 Awaiting requested review from Mai0313

@Karthikeya-Meesala Karthikeya-Meesala Awaiting requested review from Karthikeya-Meesala

@benstein benstein Awaiting requested review from benstein

@r-b-g-b r-b-g-b Awaiting requested review from r-b-g-b

@Andrew8xx8 Andrew8xx8 Awaiting requested review from Andrew8xx8

@martinzrrl martinzrrl Awaiting requested review from martinzrrl

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sonichi @BeibinLi @PyroGenesis