Skip to content

Handle infini-gram 422s for unavailable indexes better#565

Merged
mtblanton merged 3 commits intomainfrom
handle-infini-gram-422
Mar 24, 2026
Merged

Handle infini-gram 422s for unavailable indexes better#565
mtblanton merged 3 commits intomainfrom
handle-infini-gram-422

Conversation

@mtblanton
Copy link
Copy Markdown
Contributor

@mtblanton mtblanton commented Mar 19, 2026

Copy link
Copy Markdown
Contributor

@warmbowski warmbowski left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Contributor

@bronzle bronzle left a comment

Choose a reason for hiding this comment

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

Looks good

👍 on the Problem updates

@mtblanton mtblanton merged commit 25661fa into main Mar 24, 2026
8 checks passed
@mtblanton mtblanton deleted the handle-infini-gram-422 branch March 24, 2026 20:01
mtblanton added a commit to allenai/olmo-ui that referenced this pull request Mar 24, 2026
This handles the Problem responses coming from the FastAPI API in the
OlmoTrace part of the UI. I think this may be the only API call we have
that isn't going through react query now?

API PR: allenai/olmo-api#565

<img width="912" height="910" alt="image"
src="https://github.com/user-attachments/assets/1e1e5141-0d3d-4ad1-8354-9c3abcc9273b"
/>
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