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

added ctx_size parameter #148

Merged
merged 3 commits into from
Mar 15, 2023
Merged

Conversation

RazeLighter777
Copy link
Contributor

Adds a parameter called context size (-c for short) that allows taking the context size from the user's input.

Defaults to the same hardcoded 512.

@RazeLighter777
Copy link
Contributor Author

Usage -c N, --ctx_size N

@eous
Copy link

eous commented Mar 15, 2023

#71 (comment) feel free to take a look at this diff, theres probably two other lines you want to change. Probably ignore the EPS stuff though that should be changed too at some point.

@RazeLighter777
Copy link
Contributor Author

Not 100% sure if this is right but I added it.

utils.h Show resolved Hide resolved
utils.cpp Outdated Show resolved Hide resolved
main.cpp Outdated Show resolved Hide resolved
main.cpp Outdated Show resolved Hide resolved
@ggerganov ggerganov merged commit 2d64715 into ggerganov:master Mar 15, 2023
blackhole89 pushed a commit that referenced this pull request Mar 15, 2023
* added ctx_size parameter

* added it in more places

* Apply suggestions from code review

---------

Co-authored-by: Georgi Gerganov <[email protected]>
bitRAKE pushed a commit to bitRAKE/llama.cpp that referenced this pull request Mar 17, 2023
* added ctx_size parameter

* added it in more places

* Apply suggestions from code review

---------

Co-authored-by: Georgi Gerganov <[email protected]>
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.

3 participants