-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Optimize System Message #84
Conversation
## Ship Names | ||
Users may use shortened names of ships (e.g. "prom" for Prometheus, or "sov" for Sovereign): | ||
Examine the user's message carefully and ask for clarification if your context contains seemingly random or otherwise unrelated information or you are unsure what exact ship they are referring to. |
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.
Would this belong here or in the section above? I don't really know, I don't do prompt optimizing, but it seems out of place
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.
It has already been stated in sections above, I'm just repeating it here as even if the ai figures out what the user means by a certain abbreviation, the actual wiki pages the model is given may be unrelated and useless.
(It's an attempt to address the issue you referred to in your other comment.)
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.
stuff i left on conversation.txt also apply to oneoff.txt where applicable
but none of the stuff i said are necessarily requirements so i'll leave them as comments
Have you tested this and ensured no breakage? If not, I'll probably end up having to do that. |
I do not have a dev environment set up at the moment, but it should work without issue considering it is just loading an updated text file. |
i'll clone this branch and run some quick checks before merging |
In the future please don't PR your main branch, its a pain to set up 😖 |
okay in my findings:
there doesnt seem to be any lost functionality though, so i'll merge this in and accept follow up pr's addressing these issues |
This is likely something that will be continually iterated on and tested.
This version is formatted in a similar manner to how OpenAI formats their system messages. It also includes more specific instructions and some common abbreviations.
As a side-note, the distinction between a single question and a conversation is unnecessary, it's all the same to the model. As such, specifically stating in the system prompt that it will be given a conversation is not needed.