Skip to content

Claude SUTs work - #1635

Merged
bkorycki merged 2 commits into
mainfrom
claude-works-temp
Sep 2, 2026
Merged

bkorycki merged 2 commits into
mainfrom
claude-works-temp

Conversation

@bkorycki

@bkorycki bkorycki commented Sep 1, 2026

Copy link
Copy Markdown
Contributor
  • If the version is > 4, ignore temperature
  • Ignore any returned thinking blocks

@bkorycki
bkorycki requested a review from a team as a code owner September 1, 2026 20:12
@bkorycki
bkorycki deployed to Scheduled Testing September 1, 2026 20:12 — with GitHub Actions Active
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@wpietri wpietri 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.

Makes sense, and I like how clear the tests are.

@dhosterman dhosterman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@bkorycki
bkorycki merged commit c234580 into main Sep 2, 2026
3 checks passed

This branch was successfully deployed

1 active deployment
Scheduled Testing — 66b252fb Deployed Sep 1, 2026 by bkorycki via build #2513
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