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

feat(models) Set new context limits (CODY-5022) #7556

Merged
merged 4 commits into from
Mar 28, 2025
Merged

Conversation

julialeex
Copy link
Contributor

@julialeex julialeex commented Mar 26, 2025

New feature flag enhanced-context-window to increase the limits.

Update the limits (controlled by the flag) based the values in this table.

Server-side PR: https://github.com/sourcegraph/sourcegraph/pull/4321

Test plan

  1. Test all the test cases listed here

Add tests to model.test.ts and sync.test.ts

  1. Test the feature flag and context window changes e2e locally (see server side PR for more details)

@julialeex julialeex force-pushed the jlxu/new_context_limit branch from 24af342 to 5a0799b Compare March 26, 2025 04:02
@julialeex julialeex changed the title wip wip (CODY-5022) Mar 26, 2025
@julialeex julialeex requested a review from abeatrix March 26, 2025 04:10
@julialeex julialeex force-pushed the jlxu/new_context_limit branch from 5a0799b to 07ba214 Compare March 26, 2025 04:34
add new limits

update tests

wip
@julialeex julialeex force-pushed the jlxu/new_context_limit branch from 07ba214 to b0585f0 Compare March 26, 2025 18:12
@julialeex julialeex changed the title wip (CODY-5022) feat(models) (CODY-5022) Mar 26, 2025
@julialeex julialeex marked this pull request as ready for review March 26, 2025 18:26
@julialeex julialeex requested review from valerybugakov and a team March 28, 2025 03:19
@julialeex julialeex changed the title feat(models) (CODY-5022) feat(models) Set new context limits (CODY-5022) Mar 28, 2025
@julialeex julialeex enabled auto-merge (squash) March 28, 2025 16:33
@julialeex julialeex merged commit dd10bea into main Mar 28, 2025
20 of 22 checks passed
@julialeex julialeex deleted the jlxu/new_context_limit branch March 28, 2025 16:48
abeatrix pushed a commit that referenced this pull request Mar 28, 2025
New feature flag `enhanced-context-window` to increase the limits.

Update the limits (controlled by the flag) based the values in [this
table](https://docs.google.com/spreadsheets/d/166KyI-B3SxClFF8CXmGYWMUrGdVBoFaCrWtveLUkP4I/edit?gid=0#gid=0).

Server-side PR: sourcegraph/sourcegraph#4321

## Test plan
1. Test all the test cases listed
[here](https://docs.google.com/spreadsheets/d/166KyI-B3SxClFF8CXmGYWMUrGdVBoFaCrWtveLUkP4I/edit?gid=194443342#gid=194443342)

Add tests to `model.test.ts` and `sync.test.ts`

2. Test the feature flag and context window changes e2e locally (see
server side PR for more details)
- `sg start`
- `sg start cody-gateway`
- check console logs
- Loom: https://www.loom.com/share/00c66bd7e24f4b9788e5a5ec51cce05d


 <br> Backport dd10bea from #7556

Co-authored-by: julialeex <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants