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

Is there any way to adjust parameters like -c, -b, --mlock, --min-p, --top-p, --top-k, --repeat-penalty, --no-penalize-nl and --temp etc. in api_like_OAI.py #5766

Closed
SeriousHulk opened this issue Feb 28, 2024 · 4 comments · Fixed by #5808

Comments

@SeriousHulk
Copy link

          Is there any way to adjust parameters like -c, -b, --mlock, --min-p, --top-p, --top-k, --repeat-penalty, --no-penalize-nl and --temp etc. in api_like_OAI.py

Originally posted by @SeriousHulk in #4367 (comment)

@ggerganov
Copy link
Owner

Slightly off-topic, but what does api_like_OAI.py currently offer that server does not?

I was thinking of removing that script since I believe server already support the OAI API

@nelsonhurstdev
Copy link

IMO, It can be removed. It is redundant now that the server has it integrated.

@Azeirah
Copy link
Contributor

Azeirah commented Feb 29, 2024

Slightly off-topic, but what does api_like_OAI.py currently offer that server does not?

I was thinking of removing that script since I believe server already support the OAI API

I don't think it offers anything extra anymore. Only thing I do know is that even today many people (I see it on reddit /r/LocalLLama and on LLM discords) don't know that the built-in server supports OAI endpoints.

Perhaps it shouldn't be removed but just give a big fat warning "PLEASE MOVE TO ./server THIS SCRIPT WILL BE REMOVED AT x DATE"

@AFKler
Copy link

AFKler commented Mar 1, 2024

Slightly off-topic, but what does api_like_OAI.py currently offer that server does not?

I was thinking of removing that script since I believe server already support the OAI API

server does not support the following arguments: --chat-prompt --user-name --ai-name --system-name
and README has not enough information on how to use --chat-template JINJA_TEMPLATE instead.

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 a pull request may close this issue.

5 participants