Conversation
Contributor
📝 WalkthroughWalkthroughThe Hugging Face comprehensive test now references the Novita provider path for the Llama vision model. No other test configuration or logic changes. ChangesHugging Face test update
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
This was referenced Jul 28, 2026
Member
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
sammaji
force-pushed
the
07-28-fix_replaces_deprecated_huggingface_model_from_core_tests
branch
from
July 28, 2026 11:12
df5f829 to
237e43f
Compare
sammaji
force-pushed
the
07-28-fixes_skipped_framework_tests
branch
from
July 28, 2026 11:12
e267a1f to
50d76e0
Compare
sammaji
marked this pull request as ready for review
July 28, 2026 11:13
sammaji
force-pushed
the
07-28-fixes_skipped_framework_tests
branch
from
July 28, 2026 11:34
50d76e0 to
61dc665
Compare
sammaji
force-pushed
the
07-28-fix_replaces_deprecated_huggingface_model_from_core_tests
branch
from
July 28, 2026 11:34
237e43f to
f8346c9
Compare
akshaydeo
previously approved these changes
Jul 29, 2026
Contributor
Merge activity
|
akshaydeo
changed the base branch from
07-28-fixes_skipped_framework_tests
to
graphite-base/5615
July 29, 2026 07:51
akshaydeo
dismissed stale reviews from coderabbitai[bot] and themself
July 29, 2026 07:51
The base branch was changed.
akshaydeo
deleted the
07-28-fix_replaces_deprecated_huggingface_model_from_core_tests
branch
July 29, 2026 07:52
akhsaul
pushed a commit
to akhsaul/bifrost
that referenced
this pull request
Aug 27, 2026
## Summary The vision model provider for HuggingFace tests has been switched from `groq` to `novita` for the Llama-4-Scout-17B-16E-Instruct model, likely due to availability or reliability issues with the `groq` provider for this model. ## Changes - Replaced `groq/meta-llama/Llama-4-Scout-17B-16E-Instruct` with `novita/meta-llama/Llama-4-Scout-17B-16E-Instruct` as the vision model in the HuggingFace test configuration. ## Type of change - [x] Bug fix - [ ] Feature - [ ] Refactor - [ ] Documentation - [ ] Chore/CI ## Affected areas - [ ] Core (Go) - [ ] Transports (HTTP) - [x] Providers/Integrations - [ ] Plugins - [ ] UI (React) - [ ] Docs ## How to test ```sh go test ./core/providers/huggingface/... ``` Ensure the HuggingFace vision model test passes using the `novita` provider endpoint. ## Screenshots/Recordings N/A ## Breaking changes - [ ] Yes - [x] No ## Related issues N/A ## Security considerations No security implications. This change only affects which provider backend is used for a test model route. ## Checklist - [ ] I read `docs/contributing/README.md` and followed the guidelines - [x] I added/updated tests where appropriate - [ ] I updated documentation where needed - [x] I verified builds succeed (Go and UI) - [ ] I verified the CI pipeline passes locally if applicable
occcat
pushed a commit
to occcat/bifrost
that referenced
this pull request
Sep 2, 2026
## Summary The vision model provider for HuggingFace tests has been switched from `groq` to `novita` for the Llama-4-Scout-17B-16E-Instruct model, likely due to availability or reliability issues with the `groq` provider for this model. ## Changes - Replaced `groq/meta-llama/Llama-4-Scout-17B-16E-Instruct` with `novita/meta-llama/Llama-4-Scout-17B-16E-Instruct` as the vision model in the HuggingFace test configuration. ## Type of change - [x] Bug fix - [ ] Feature - [ ] Refactor - [ ] Documentation - [ ] Chore/CI ## Affected areas - [ ] Core (Go) - [ ] Transports (HTTP) - [x] Providers/Integrations - [ ] Plugins - [ ] UI (React) - [ ] Docs ## How to test ```sh go test ./core/providers/huggingface/... ``` Ensure the HuggingFace vision model test passes using the `novita` provider endpoint. ## Screenshots/Recordings N/A ## Breaking changes - [ ] Yes - [x] No ## Related issues N/A ## Security considerations No security implications. This change only affects which provider backend is used for a test model route. ## Checklist - [ ] I read `docs/contributing/README.md` and followed the guidelines - [x] I added/updated tests where appropriate - [ ] I updated documentation where needed - [x] I verified builds succeed (Go and UI) - [ ] I verified the CI pipeline passes locally if applicable
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
The vision model provider for HuggingFace tests has been switched from
groqtonovitafor the Llama-4-Scout-17B-16E-Instruct model, likely due to availability or reliability issues with thegroqprovider for this model.Changes
groq/meta-llama/Llama-4-Scout-17B-16E-Instructwithnovita/meta-llama/Llama-4-Scout-17B-16E-Instructas the vision model in the HuggingFace test configuration.Type of change
Affected areas
How to test
go test ./core/providers/huggingface/...Ensure the HuggingFace vision model test passes using the
novitaprovider endpoint.Screenshots/Recordings
N/A
Breaking changes
Related issues
N/A
Security considerations
No security implications. This change only affects which provider backend is used for a test model route.
Checklist
docs/contributing/README.mdand followed the guidelines