-
Notifications
You must be signed in to change notification settings - Fork 35
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
Removal of context from PHP code #3
Comments
Hi! In that commit I updated the code to use the new ChatGPT API ( You can watch me make these changes in this video. After this commit, you can set a system message in the |
Thanks @unconv cleared that up for me! I have two more questions.
|
I think there can only be one system message. Not sure why you would like to add more than one, anyway. You can always add more content to the system message. There does not seem to be much difference between setting a system message and just sending a regular message, though. So you can change the style of answer also by your own messages (which is also why you need to worry about "prompt injection") I haven't tried fine-tuning yet, but it does sound interesting. When I do, I might add a possibility to use the fine-tuning models in the settings. Good suggestion! |
Hi @unconv I noticed that in the following commit 066c33b you removed the context from the code in favour of using simply $msg as the context.
Was there something wrong with the old way?
How can I best provide context for the PHP app as I want users to be primed with some context.
The text was updated successfully, but these errors were encountered: