Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add /sc cmd for chat #2577

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.

Sign up for GitHub

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

Jump to bottom
Closed
reidliu41 wants to merge 1 commit into from
Closed

add /sc cmd for chat #2577

reidliu41 wants to merge 1 commit into from
+73 −0

Conversation

@reidliu41
Copy link
Contributor

@reidliu41 reidliu41 commented Nov 4, 2024 •
edited
Loading

  • Only the fixed default, cli_helper context, so add /sc cmd to customize the context
>>> /h                                                                                  [S][default]
╭───────────────────────────────────────────── system ─────────────────────────────────────────────╮
│ Help / TL;DR                                                                                     │
│                                                                                                  │
│  • /q: quit                                                                                      │
│  • /h: show help                                                                                 │
│  • /a assistant: amend assistant (i.e., model)                                                   │
│  • /c context: change context (available contexts: default, cli_helper)                          │
│  • /lc: list contexts                                                                            │
│  • /m: toggle multiline (for the next session only)                                              │
│  • /M: toggle multiline                                                                          │
│  • /n: new session                                                                               │
│  • /N: new session (ignoring loaded)                                                             │
│  • /d <int>: display previous response based on input, if passed 1 then previous, if 2 then      │
│    second last response and so on.                                                               │
│  • /p <int>: previous response in plain text based on input, if passed 1 then previous, if 2     │
│    then second last response and so on.                                                          │
│  • /md <int>: previous response in Markdown based on input, if passed 1 then previous, if 2 then │
│    second last response and so on.                                                               │
│  • /s filepath: save current session to filepath                                                 │
│  • /l filepath: load filepath and start a new session                                            │
│  • /L filepath: load filepath (permanently) and start a new session                              │
│  • /sc context_name custom_context_text: set custom context with context_name and                │
│    custom_context_text.                                                                          │
│     • Note: For multi-word context_name, use underscores (_) to separate words.                  │
│                                                                                                  │
│ Press Alt (or Meta) and Enter or Esc Enter to end multiline input.                               │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
>>> /sc test_context Your name is test-ilab.                                            [S][default]
╭───────────────────────────────────────────── system ─────────────────────────────────────────────╮
│ Welcome to InstructLab Chat w/ MERLINITE-7B-LAB-Q4_K_M.GGUF (new session)                        │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
╭───────────────────────────────────────────── system ─────────────────────────────────────────────╮
│ INFO: Custom context test_context added and activated.                                           │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
>>> what is your name?                                                             [S][test_context]
╭────────────────────────────────── merlinite-7b-lab-Q4_K_M.gguf ──────────────────────────────────╮
│ test-ilab.                                                                                       │
╰────────────────────────────────────────────────────────────────────────── elapsed 0.185 seconds ─╯
>>> exit                                                                           [S][test_context]

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

@mergify mergify bot added testing Relates to testing ci-failure PR has at least one CI failure labels Nov 4, 2024
@mergify mergify bot removed the ci-failure PR has at least one CI failure label Nov 4, 2024
@nathan-weinberg nathan-weinberg requested a review from a team November 4, 2024 14:59
@reidliu41
Copy link
Contributor Author

reidliu41 commented Nov 4, 2024 •
edited
Loading

hi team, anyone can help to take a look? +1

jaideepr97
jaideepr97 reviewed Nov 8, 2024
View reviewed changes
Copy link
Member

@jaideepr97 jaideepr97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmmm, contexts are closely tied to what system prompt gets used for the model
I think this is veering into the domain of supporting bring-your-own-system prompts. This is an area we do we want to support in the future, but might need some research on how we want to do this
would like to get @Maxusmusti's input on this

@jaideepr97 jaideepr97 mentioned this pull request Nov 14, 2024
Closed
6 tasks
@nathan-weinberg
Copy link
Member

nathan-weinberg commented Jan 10, 2025

@Maxusmusti could you PTAL?

booxter
booxter suggested changes Feb 18, 2025
View reviewed changes
@mergify
Copy link
Contributor

mergify bot commented Feb 18, 2025

This pull request has merge conflicts that must be resolved before it can be merged. @reidliu41 please rebase it. https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify bot added the needs-rebase This Pull Request needs to be rebased label Feb 18, 2025
@mergify mergify bot added ci-failure PR has at least one CI failure and removed needs-rebase This Pull Request needs to be rebased labels Feb 19, 2025
@reidliu41
Copy link
Contributor Author

reidliu41 commented Feb 19, 2025

doc issue: #3169

@mergify mergify bot removed the ci-failure PR has at least one CI failure label Mar 5, 2025
@reidliu41 reidliu41 requested review from booxter and removed request for Maxusmusti March 5, 2025 22:17
booxter
booxter suggested changes Mar 6, 2025
View reviewed changes
@reidliu41 reidliu41 force-pushed the add-sc-for-chat branch 2 times, most recently from 0699311 to 6eea596 Compare March 7, 2025 03:30
@mergify mergify bot added the ci-failure PR has at least one CI failure label Mar 7, 2025
@reidliu41
feat: add /sc cmd for chat
a7e05ca 
- help to add custom context

Signed-off-by: reid_liu <[email protected]>
@mergify mergify bot removed the ci-failure PR has at least one CI failure label Mar 7, 2025
@mergify mergify bot added the one-approval PR has one approval from a maintainer label Mar 7, 2025
@github-actions
Copy link

github-actions bot commented May 18, 2025

This pull request has been automatically marked as stale because it has not had activity within 60 days. It will be automatically closed if no further activity occurs within 30 days.

@github-actions github-actions bot added the stale label May 18, 2025
@mergify
Copy link
Contributor

mergify bot commented May 18, 2025

This pull request has merge conflicts that must be resolved before it can be merged. @reidliu41 please rebase it. https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify bot added the needs-rebase This Pull Request needs to be rebased label May 18, 2025
@github-actions github-actions bot removed the stale label May 19, 2025
@github-actions
Copy link

github-actions bot commented Jul 18, 2025

This pull request has been automatically marked as stale because it has not had activity within 60 days. It will be automatically closed if no further activity occurs within 30 days.

@github-actions github-actions bot added the stale label Jul 18, 2025
@mergify mergify bot removed the one-approval PR has one approval from a maintainer label Jul 18, 2025
@github-actions github-actions bot removed the stale label Jul 19, 2025
@github-actions
Copy link

github-actions bot commented Sep 18, 2025

This pull request has been automatically marked as stale because it has not had activity within 60 days. It will be automatically closed if no further activity occurs within 30 days.

@github-actions github-actions bot added the stale label Sep 18, 2025
@github-actions
Copy link

github-actions bot commented Oct 19, 2025

This pull request has been automatically closed due to inactivity. Please feel free to reopen if you intend to continue working on it!

@github-actions github-actions bot closed this Oct 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jaideepr97 jaideepr97 jaideepr97 left review comments

+1 more reviewer

@booxter booxter booxter approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

needs-rebase This Pull Request needs to be rebased stale testing Relates to testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@reidliu41 @nathan-weinberg @booxter @jaideepr97