Skip to content

Conversation

martindevans
Copy link
Member

Modified ContextSize in parameters to be nullable. A null value means autodetect from the model.

Changed default value from 512 to null

@SanftMonster
Copy link
Collaborator

Hey, do you still have the accessability to merge the PRs? I added master branch protection yesterday but am not sure its effects.

@martindevans
Copy link
Member Author

I have a UI like this:

firefox_2023-11-08_14-55-38

Since I can review code I can also merge it.

I didn't immediately merge this one since that would either require me to review my own code or to click the scary "bypass" checkbox 😆

@martindevans martindevans merged commit 74a8ca5 into SciSharp:master Nov 8, 2023
@martindevans martindevans deleted the context_autodetect branch November 8, 2023 14:58
@SanftMonster
Copy link
Collaborator

Okay :) Please feel free to review/merge PRs. I added the protection of master just to avoid forced-push by mistake.

martindevans added a commit to martindevans/LLamaSharp that referenced this pull request Nov 15, 2023
…hat it shouldn't. Fixed that.

Originally from these PRs:
 - SciSharp#263
 - SciSharp#259
@martindevans martindevans mentioned this pull request Nov 15, 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

Successfully merging this pull request may close these issues.

2 participants