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

Allow for prepending (instead of appending) context to the system message #414

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tschwinge
Copy link

... via new 'system-prepend'.

Certain use cases/LLMs may prefer to first see the context, then the system message.

@tschwinge
Copy link
Author

This textually depends on #413 'Consistently use "last user prompt" for where context may be included' (and therefore includes those changes in this PR here).

@tschwinge
Copy link
Author

I'll gladly take any pointer about how to re-do gptel--infix-use-context in a more "visually pleasing" way.

…sage

... via new 'system-prepend'.

Certain use cases/LLMs may prefer to first see the context, then the system
message.
@tschwinge tschwinge force-pushed the context_system-prepend branch from fb50736 to 0d06a15 Compare October 15, 2024 15:03
@tschwinge
Copy link
Author

This textually depends on #413 'Consistently use "last user prompt" for where context may be included' (and therefore includes those changes in this PR here).

I've removed this with the last force-push.

(Not yet properly tested.)

@karthink
Copy link
Owner

@tschwinge Thanks for the PR. I've exhausted my available time for gptel fixing some bugs in the last release, so it'll be a few days before I can look at this.

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