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

Title generation isn't included in calculated costs #330

Closed
Xtrah opened this issue Jun 5, 2023 · 3 comments
Closed

Title generation isn't included in calculated costs #330

Xtrah opened this issue Jun 5, 2023 · 3 comments

Comments

@Xtrah
Copy link
Contributor

Xtrah commented Jun 5, 2023

Title generation isn't included in calculated costs (or it might be incorrectly calculated).

Tested in #329 when using Azure endpoint with GPT-4, considering that title generation uses gpt-3.5-turbo, it should incur a small charge on the gpt-3.5-turbo estimation when generating a thread title, but it currently doesn't.

@ztjhz
Copy link
Owner

ztjhz commented Jun 6, 2023

The total token costs are updated here: https://github.com/ztjhz/BetterChatGPT/blob/main/src/hooks/useSubmit.ts#L186

@Xtrah
Copy link
Contributor Author

Xtrah commented Jun 6, 2023

Considering the title generation uses gpt-3.5-turbo, shouldn't there be a gpt-3.5-turbo charge calculated here then, or does it add onto the estimated gpt-4 cost?

image

@ztjhz
Copy link
Owner

ztjhz commented Jun 6, 2023

This is a bug, currently it adds the cost to the current model chosen.

@ztjhz ztjhz closed this as completed in 7bf6f7a Jun 7, 2023
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

No branches or pull requests

2 participants