Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

DO-NOT-MERGE testing ctx limit#1190

Closed
markstur wants to merge 3 commits intoinstructlab:maininstructlab/instructlab:mainfrom
markstur:DNM-testing-ctx-limitmarkstur/instructlab:DNM-testing-ctx-limitCopy head branch name to clipboard
Closed

DO-NOT-MERGE testing ctx limit#1190
markstur wants to merge 3 commits intoinstructlab:maininstructlab/instructlab:mainfrom
markstur:DNM-testing-ctx-limitmarkstur/instructlab:DNM-testing-ctx-limitCopy head branch name to clipboard

Conversation

@markstur
Copy link
Contributor

Changes

Which issue is resolved by this Pull Request:
Resolves #

Description of your changes:

alimaredia and others added 3 commits May 17, 2024 11:05
Signed-off-by: Ali Maredia <amaredia@redhat.com>
Signed-off-by: Ali Maredia <amaredia@redhat.com>
* Trying to shrink the running context in a way that llama-cpp-python 2.75 will allow
* replacing old messages with ""
* eventually using reset
* hard-coded 100 limit hack (was testing with max 255)
* has a bug with a single long sentence
@mergify mergify bot added testing Relates to testing ci-failure PR has at least one CI failure labels May 17, 2024
@nathan-weinberg nathan-weinberg added the hold In-progress PR. Tag should be removed before merge. label May 18, 2024
@alimaredia
Copy link
Contributor

Closed for: #1161

@alimaredia alimaredia closed this May 22, 2024
@nathan-weinberg nathan-weinberg removed the hold In-progress PR. Tag should be removed before merge. label May 22, 2024
jgato pushed a commit to jgato/instructlab that referenced this pull request Jun 21, 2024
)

Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.6
to 4.1.7.

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-failure PR has at least one CI failure testing Relates to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.