-
Notifications
You must be signed in to change notification settings - Fork 929
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
Improve model #4253
Improve model #4253
Conversation
yann300
commented
Nov 19, 2023
- fix stripping the in-browser model result.
- use remote llm for code generation
✅ Deploy Preview for remixproject ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
86b66da
to
26de2f2
Compare
86fc95d
to
a6fb3a9
Compare
615696c
to
23636d4
Compare
// abort if there is a signal | ||
if (token.isCancellationRequested) { | ||
console.log('aborted') | ||
return | ||
} | ||
|
||
console.log(word) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to keep this log?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's done
// the generated text remove a space from the context. that why we need to remove all the spaces | ||
const clean = generatedText.replace(/ /g, '').replace(word.replace(/ /g, ''), '') | ||
// the generated text remove a space from the context... | ||
const clean = generatedText.replace('@custom:dev-run-script', '@custom:dev-run-script ').replace(word, '') | ||
console.log('suggest result', clean) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's done
console.log('suggest result', clean) | ||
const item: monacoTypes.languages.InlineCompletion = { | ||
insertText: clean | ||
}; | ||
|
||
// abort if there is a signal | ||
if (token.isCancellationRequested) { | ||
console.log('aborted') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one yes, this other not.
23636d4
to
83d5903
Compare
2c91f37
to
0c974e0
Compare
change url of gpt endpoint
0c974e0
to
bdd386b
Compare
76c3b6c
to
4411dc2
Compare