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

DeepCody: new model UI group #5950

Merged
merged 1 commit into from
Oct 18, 2024
Merged

DeepCody: new model UI group #5950

merged 1 commit into from
Oct 18, 2024

Conversation

abeatrix
Copy link
Contributor

CLOSE https://linear.app/sourcegraph/issue/MKT-344

Based on the discussion in the linear issue, we would like to avoid listing Deep Cody under any existing model group but on its own under 'mixed models":

  • Add a new 'DeepCody' model UI group to the ModelUIGroup object
  • Update the getModelDropDownUIGroup function to handle the 'deep-cody' model ID and return the new 'DeepCody' group

Test plan

Small UI change:

image

Changelog

- Add a new 'DeepCody' model UI group to the `ModelUIGroup` object
- Update the `getModelDropDownUIGroup` function to handle the 'deep-cody' model ID and return the new 'DeepCody' group
@abeatrix abeatrix merged commit 7c745ae into main Oct 18, 2024
19 of 20 checks passed
@abeatrix abeatrix deleted the bee/deep-cody-group branch October 18, 2024 20:29
Copy link

@aramaraju aramaraju left a comment

Choose a reason for hiding this comment

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

I don't think we should do this - check Linear for my reason

aramaraju added a commit that referenced this pull request Oct 19, 2024
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.

3 participants