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

Set Leo default model to Llama 3 8b (25% rollout) #1225

Merged
merged 5 commits into from
Oct 28, 2024
Merged

Conversation

nvonpentz
Copy link
Member

@nvonpentz nvonpentz commented Oct 15, 2024

Related to brave/brave-core#21398
Blocked on deploying https://github.com/brave/aichat-ops/pull/359 to prod

This sets the default model for 25% of free users to chat-basic, which corresponds to Llama 3 8b.

The intention is to start this to progressively roll out this change (25%, 50%, 75%, 100%), so we can ensure our single instance of Llama 3 can handle the increase in traffic. Based on these estimations, one instance of llama 3 (one gpu) should have higher token throughput than our single instance of mixtral (four gpus), so we expect this to work with two llama instances which will be added in https://github.com/brave/aichat-ops/pull/359.

cc @petemill @LorenzoMinto

Note:

Copy link
Contributor

github-actions bot commented Oct 15, 2024

✅ Test Seed Generated Successfully

To apply the test seed:

  1. Desktop: Launch the browser with --variations-pr=1225.
    Android: Set the command line to --variations-pr=1225 in debug menu, restart the browser.
    iOS: Set Variations PR to 1225 in Brave Core Switches debug menu, restart the browser.
  2. Wait 5-10 seconds to fetch the seed.
  3. Restart the browser to apply the seed.
  4. Ensure Active Variations section at brave://version starts with the expected seed version (see below).

Seed Details

Parameter Value
Version pull/1225@29f327ca26bca5d1900801196a3db1b3b478798d
Uploaded Mon, 28 Oct 2024 12:29:09 GMT
PR commit 646922f
Base commit a3ff5bd
Merge commit 29f327c
Serial number a00f6077f3eac1b09f5be572757f6ba0

seed/seed.json Outdated
"BETA",
"NIGHTLY"
],
"min_version": "122.0.6261.57",
Copy link
Member

Choose a reason for hiding this comment

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

looks like in this file the versioning is [chrome_major].[brave_version], e.g 122.1.63.0?

Copy link
Member Author

Choose a reason for hiding this comment

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

I updated to use 122.1.63.161 as the min version of the default model change. 1.63.161 seems to be the first 1.63 release. This also adds in a little buffer because brave/brave-browser#34721 was uplifted into 1.62.x.

I specified 122.1.63.160 as the new max version for the previous setting. This may not correlate to an actual release, it's just one patch version behind 122.1.63.161.

@petemill
Copy link
Member

I tested running with --enable-features=AIChat:default_model\/chat-basic and all seems fine

@nvonpentz nvonpentz marked this pull request as ready for review October 17, 2024 16:38
@nvonpentz nvonpentz requested a review from a team as a code owner October 17, 2024 16:38
@nvonpentz nvonpentz requested a review from petemill October 17, 2024 16:39
@kjozwiak
Copy link
Member

@petemill will grab this tomorrow/verify and push it into production 👍

seed/seed.json Outdated Show resolved Hide resolved
@goodov goodov force-pushed the leo-default-model branch from 9d410b5 to 646922f Compare October 28, 2024 12:27
@kjozwiak
Copy link
Member

kjozwiak commented Oct 28, 2024

Windows x64 verification(s):

Verification PASSED on Win 11 x64 using the following build(s):

Brave | 1.71.118 Chromium: 130.0.6723.70 (Official Build) (64-bit)
-- | --
Revision | d29310b051bde8ba041cd7b4edcb41ce951be12c
OS | Windows 11 Version 23H2 (Build 22631.4317)

Release - BraveAIChatDefaultModelStudy:DefaultLlama

  • launched/re-launched 1.71.118 Chromium: 130.0.6723.70 using brave.exe --enable-logging=stderr --variations-pr=1225
  • ensured that pull/1225@02be91adb07839f2668690152f397b0a9ec4c0c2 via brave://version under Active variations
  • ensured that BraveAIChatDefaultModelStudy:DefaultLlama via brave://version
  • ensured that Llama 3.1 8B is selected as the default model via brave://settings/leo-assistant
  • ensured that Llama 3.1 8B is selected as the default model via the side panel
  • ensured that Llama 3.1 8B is being used when using Leo when BraveAIChatDefaultModelStudy:DefaultLlama
Example Example Example Example
image image image image

Release - BraveAIChatDefaultModelStudy:DefaultMixtral

  • launched/re-launched 1.71.118 Chromium: 130.0.6723.70 using brave.exe --enable-logging=stderr --variations-pr=1225
  • ensured that pull/1225@02be91adb07839f2668690152f397b0a9ec4c0c2 via brave://version under Active variations
  • ensured that BraveAIChatDefaultModelStudy:DefaultMixtral via brave://version
  • ensured that Mixtral is selected as the default model via brave://settings/leo-assistant
  • ensured that Mixtral is selected as the default model via the side panel
  • ensured that Mixtral is being used when using Leo when BraveAIChatDefaultModelStudy:DefaultMixtral
Example Example Example Example
image image image image

Android verification(s):

Verification PASSED on Pixel 6 running Android 15 using the following build(s):

Brave | 1.71.118 Chromium: 130.0.6723.70 (Official Build) (64-bit)
--- | ---
Revision | d29310b051bde8ba041cd7b4edcb41ce951be12c
OS | Android 15; Build/AP41.240823.009; 35; REL

Release - BraveAIChatDefaultModelStudy:DefaultLlama

  • launched 1.71.118 Chromium: 130.0.6723.70 and added --variations-pr=1225 into the command line via QA dev tools
  • re-launched 1.71.118 Chromium: 130.0.6723.70 for the second time
  • ensured that pull/1225@02be91adb07839f2668690152f397b0a9ec4c0c2 via brave://version under Active variations
  • ensured that BraveAIChatDefaultModelStudy:DefaultLlama via brave://version
  • ensured that Llama 3.1 8B is selected as the default model via Settings -> Leo -> Default model for new conversations
  • ensured that Llama 3.1 8B is selected as the default model via Leo panel
  • ensured that Llama 3.1 8B is being used when using Leo when BraveAIChatDefaultModelStudy:DefaultLlama
Example Example Example Example Example
1000002278 1000002279 1000002280 1000002281 1000002282

Release - BraveAIChatDefaultModelStudy:DefaultMixtral

  • launched 1.71.118 Chromium: 130.0.6723.70 and added --variations-pr=1225 into the command line via QA dev tools
  • re-launched 1.71.118 Chromium: 130.0.6723.70 for the second time
  • ensured that pull/1225@02be91adb07839f2668690152f397b0a9ec4c0c2 via brave://version under Active variations
  • ensured that BraveAIChatDefaultModelStudy:DefaultMixtral via brave://version
  • ensured that Mixtral is selected as the default model via Settings -> Leo -> Default model for new conversations
  • ensured that Mixtral is selected as the default model via Leo panel
  • ensured that Mixtral is being used when using Leo when BraveAIChatDefaultModelStudy:DefaultMixtral
Example Example Example Example Example
Screenshot_20241028-123313 Screenshot_20241028-123336 Screenshot_20241028-123339 Screenshot_20241028-123405 Screenshot_20241028-123443

iOS verification(s):

Verification PASSED on iPhone 11 running iOS 18 using the following build(s):

Brave | 1.70.133 Chromium: 129.0.6668.100 (Official Build) stable (64-bit) 
--- | ---
Revision | 53596529e7d7...
OS | iOS

Release - BraveAIChatDefaultModelStudy:DefaultLlama

  • launched 1.70.133 Chromium: 129.0.6668.100 and added 1225 into the Variations PR switch under BraveCore Switches
  • re-launched 1.70.133 Chromium: 129.0.6668.100 for the second time
  • ensured that pull/1225@02be91adb07839f2668690152f397b0a9ec4c0c2 via brave://version under Active variations
  • ensured that BraveAIChatDefaultModelStudy:DefaultLlama via brave://version
  • ensured that Llama 3.1 8B is selected as the default model via Settings -> Leo -> Default model
  • ensured that Llama 3.1 8B is selected as the default model via Leo panel
  • ensured that Llama 3.1 8B is being used when using Leo when BraveAIChatDefaultModelStudy:DefaultLlama
Example Example Example Example Example Example
IMG_0615 IMG_0616 IMG_0617 IMG_0618 IMG_0619 IMG_0620

Release - BraveAIChatDefaultModelStudy:DefaultMixtral

  • launched 1.70.133 Chromium: 129.0.6668.100 and added 1225 into the Variations PR switch under BraveCore Switches
  • re-launched 1.70.133 Chromium: 129.0.6668.100 for the second time
  • ensured that pull/1225@02be91adb07839f2668690152f397b0a9ec4c0c2 via brave://version under Active variations
  • ensured that BraveAIChatDefaultModelStudy:DefaultMixtral via brave://version
  • ensured that Mixtral is selected as the default model via Settings -> Leo -> Default model
  • ensured that Mixtral is selected as the default model via Leo panel
  • ensured that Mixtral is being used when using Leo when BraveAIChatDefaultModelStudy:DefaultMixtral
Example Example Example Example Example Example
IMG_0622 IMG_0623 IMG_0624 IMG_0626 IMG_0625 IMG_0627

Copy link
Member

@kjozwiak kjozwiak left a comment

Choose a reason for hiding this comment

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

Uplift into production approved after deliberating with @brave/uplift-approvers.

@kjozwiak kjozwiak added this pull request to the merge queue Oct 28, 2024
Merged via the queue into main with commit 619d91e Oct 28, 2024
6 checks passed
@kjozwiak kjozwiak deleted the leo-default-model branch October 28, 2024 18:22
github-merge-queue bot pushed a commit that referenced this pull request Oct 30, 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.

4 participants