Skip to content

use GitHub alerts and fix LLMS_AVAILABLE token - #1153

Merged
nl917 merged 3 commits into
mainfrom
modify-warning
Jul 29, 2026
Merged

use GitHub alerts and fix LLMS_AVAILABLE token #1153
nl917 merged 3 commits into
mainfrom
modify-warning

Conversation

@nl917

@nl917 nl917 commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

@nl917 nl917 changed the title make it a note use GitHub alerts and fix LLMS_AVAILABLE token Jul 29, 2026
@nl917
nl917 requested review from dolaameng and goeffthomas July 29, 2026 20:36
@dolaameng
dolaameng requested a review from stevemessick July 29, 2026 20:39

@dolaameng dolaameng left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

Comment thread docs/benchmarks.md

> **Note:** `LLMS_AVAILABLE` is a curated subset of models intended for local development and testing — it is **not** the full set of available models, and the Model Proxy token itself is not restricted to these models. To see all available models, use `kaggle benchmarks tasks models`. To run a task against any model (including those not in `LLMS_AVAILABLE`), use `kaggle benchmarks tasks run`, which executes on Kaggle's infrastructure with access to the full model catalog.
> [!WARNING]
> `LLMS_AVAILABLE` is a curated subset of models intended for local development and testing — it is **not** the full set of available models, and the Model Proxy token minted for local development **is** restricted to these models. To see all available models, use `kaggle benchmarks tasks models`. To run a task against any model (including those not in `LLMS_AVAILABLE`), use `kaggle benchmarks tasks run`, which executes on Kaggle's infrastructure with access to the full model catalog.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a side note: Actually users can use whatever models they choose by manually injecting them into LLMS_AVAILABLE. I don't think we have any backend restrictions for this so it's purely conventional.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

FYI, the MP token we mint does restrict the ones they can use. The local development MP token is different from the one minted during Kaggle KKB sessions

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

To illustrate, this is the MP token creation that we do when users make one via the API: https://github.com/Kaggle/kaggleazure/blob/ci/Kaggle.Services.Shared/ModelProxy/Utils/TokenUtil.cs#L61

Note that the LocalModelProxyGroup is tied to this specific subset of models that aligns with what we set for them in LLMS_AVAILABLE: https://source.corp.google.com/piper///depot/google3/cloud/kaggle/modelproxy/backends/config/config.gcl;l=2424-2440

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oh that's good to know. I guess my knowledge was outdated or never accurate.

@nl917
nl917 merged commit 5cee8fd into main Jul 29, 2026
21 checks passed
@nl917
nl917 deleted the modify-warning branch July 29, 2026 21:47
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