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

feat(agent): add experimental option: strip auto-closing chars in prompt suffix. #651

Merged

Conversation

icycodes
Copy link
Member

@icycodes icycodes commented Oct 27, 2023

Complete TAB-268

Add this config to enable experimental feature:

[completion.prompt]
stripAutoClosingCharacters = true

Example:
For this case:

function fib(n) {}

off(default):

// prompt
{
  prefix: "function fib(n) {",
  suffix: "}\n",
}

on:

// prompt
{
  prefix: "function fib(n) {",
  suffix: "\n",
}

@icycodes icycodes marked this pull request as draft October 27, 2023 11:02
@wsxiaoys
Copy link
Member

wsxiaoys commented Oct 27, 2023

change naming to experimentalStripAutoClosingCharacters

@icycodes icycodes force-pushed the feat-agent-add-option-strip-auto-closing-chars branch from 58372bd to 12efdfe Compare October 30, 2023 02:35
@icycodes icycodes marked this pull request as ready for review October 30, 2023 02:36
@wsxiaoys wsxiaoys merged commit 238d81a into TabbyML:main Oct 30, 2023
3 checks passed
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