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

Prompt shouldn't include Character limit #246

Open
moritztim opened this issue Sep 7, 2023 · 0 comments
Open

Prompt shouldn't include Character limit #246

moritztim opened this issue Sep 7, 2023 · 0 comments
Labels
feature New feature or request pending triage

Comments

@moritztim
Copy link

Feature request

Apparently LLMs might have a general understanding of roughly how much 50 characters is but since there's already a max_tokens defined, I don't think it helps to set a character limit, since it's not gonna result in more exact results than just relying on max_tokens.

Why?

It's not that big of a deal, I just thought it's an, admittedly small, waste of prompt tokens for every commit.

Alternatives

No response

Additional context

https://help.openai.com/en/articles/4936856-what-are-tokens-and-how-to-count-them

Using ShellGPT to test the character count accuracy of GPT 4

for i in {1..10}; do sgpt --chat --model=gpt-4 --no-cache 'write a sentence with exactly 50 characters' | wc -m; done 
45
53
55
56
57
53
60
57
54
57
@moritztim moritztim added feature New feature or request pending triage labels Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request pending triage
Projects
None yet
Development

No branches or pull requests

1 participant