Skip to content

Add qwen3-coder-30b-a3b-instruct model to resolve_model_config.py#1903

Merged
juanmichelini merged 1 commit intomainfrom
openhands/add-qwen3-coder-30b-a3b-instruct
Feb 4, 2026
Merged

Add qwen3-coder-30b-a3b-instruct model to resolve_model_config.py#1903
juanmichelini merged 1 commit intomainfrom
openhands/add-qwen3-coder-30b-a3b-instruct

Conversation

@juanmichelini
Copy link
Collaborator

Summary

Adds the model litellm_proxy/Qwen3-Coder-30B-A3B-Instruct with id qwen3-coder-30b-a3b-instruct to the expected models in resolve_model_config.py.

Fixes #1901

Changes

  • Added new model entry qwen3-coder-30b-a3b-instruct to the MODELS dictionary in .github/run-eval/resolve_model_config.py
  • Added the model to EXPECTED_MODELS list in tests/github_workflows/test_resolve_model_config.py

Checklist

  • If the PR is changing/adding functionality, are there tests to reflect this?
  • If there is an example, have you run the example to make sure that it works?
  • If there are instructions on how to run the code, have you followed the instructions and made sure that it works?
  • If the feature is significant enough to require documentation, is there a PR open on the OpenHands/docs repository with the same branch name?
  • Is the github CI passing?

@juanmichelini can click here to continue refining the PR

- Add new model entry with id 'qwen3-coder-30b-a3b-instruct'
- Model uses litellm_proxy/Qwen3-Coder-30B-A3B-Instruct
- Add model to EXPECTED_MODELS in test file

Fixes #1901

Co-authored-by: openhands <[email protected]>
Copy link
Collaborator

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

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

Overall, the changes look good! The model entry follows the correct structure and tests were properly updated. I have one minor observation about model path consistency.

"qwen3-coder-30b-a3b-instruct": {
"id": "qwen3-coder-30b-a3b-instruct",
"display_name": "Qwen3 Coder 30B A3B Instruct",
"llm_config": {"model": "litellm_proxy/Qwen3-Coder-30B-A3B-Instruct"},
Copy link
Collaborator

Choose a reason for hiding this comment

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

🟡 Suggestion: The model path uses a direct format (litellm_proxy/Qwen3-Coder-30B-A3B-Instruct) without a provider prefix, while other Qwen models in this file use provider-specific paths:

  • qwen-3-coder uses litellm_proxy/fireworks_ai/qwen3-coder-480b-a35b-instruct
  • qwen3-coder-next uses litellm_proxy/openrouter/qwen/qwen3-coder-next

Is Qwen3-Coder-30B-A3B-Instruct configured directly in the LiteLLM proxy, or should this follow the provider pattern like the other Qwen models? If the direct path is correct (e.g., it's a custom endpoint), then this is fine as-is.

@juanmichelini juanmichelini merged commit 62b4744 into main Feb 4, 2026
24 of 25 checks passed
@juanmichelini juanmichelini deleted the openhands/add-qwen3-coder-30b-a3b-instruct branch February 4, 2026 15:43
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.

Add litellm_proxy/Qwen3-Coder-30B-A3B-Instruct to

4 participants