Skip to content

Fix together dynamic factory#1162

Merged
bkorycki merged 2 commits into
mainfrom
fix-together-dynamic
Jul 30, 2025
Merged

Fix together dynamic factory#1162
bkorycki merged 2 commits into
mainfrom
fix-together-dynamic

Conversation

@bkorycki
Copy link
Copy Markdown
Contributor

Together's models API is broken. Roger opened an issue 2 weeks ago that has still not been acknowledged.

This is one of the issues causing smoke test failures.

This PR fixes the issue by no longer relying on the models call.

@bkorycki bkorycki requested review from rogthefrog and superdosh July 30, 2025 20:16
@bkorycki bkorycki requested a review from a team as a code owner July 30, 2025 20:16
@bkorycki bkorycki had a problem deploying to Scheduled Testing July 30, 2025 20:16 — with GitHub Actions Failure
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 30, 2025

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

Copy link
Copy Markdown
Contributor

@rogthefrog rogthefrog left a comment

Choose a reason for hiding this comment

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

Awesome.

@bkorycki bkorycki temporarily deployed to Scheduled Testing July 30, 2025 20:42 — with GitHub Actions Inactive
@bkorycki bkorycki temporarily deployed to Scheduled Testing July 30, 2025 20:42 — with GitHub Actions Inactive
@bkorycki bkorycki temporarily deployed to Scheduled Testing July 30, 2025 20:42 — with GitHub Actions Inactive
@bkorycki bkorycki merged commit ff8050a into main Jul 30, 2025
4 checks passed
@bkorycki bkorycki deleted the fix-together-dynamic branch July 30, 2025 20:45
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants