camel icon indicating copy to clipboard operation
camel copied to clipboard

add AIHubMix as a new Provider

Open xinrui-z opened this issue 1 month ago • 3 comments

Add AIHubMix as a new Provider, including:

  1. Implementation of AihubmixModel class.
  2. Example script aihubmix_model_example.py in examples/models.
  3. Updated .env.example with AIHubMix environment variables.

Fixes #3369

Checklist

  • [x] I have read the CONTRIBUTION guide (required)
  • [x] I have linked this PR to an issue using the Development section on the right sidebar or by adding Fixes #issue-number in the PR description (required)
  • [x] I have checked if any dependencies need to be added or updated in pyproject.toml and uv lock
  • [x] I have updated the tests accordingly (required for a bug fix or a new feature)
  • [x] I have updated the documentation if needed:
  • [x] I have added examples if this is a new feature

xinrui-z avatar Nov 04 '25 08:11 xinrui-z

[!IMPORTANT]

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • [ ] Create PR with unit tests
  • [ ] Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

ā¤ļø Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot] avatar Nov 04 '25 08:11 coderabbitai[bot]

@xinrui-z hi,thanks for this pr! notice that https://github.com/camel-ai/camel/pull/3338 has been merged ,Is this PR somewhat repetitive?

fengju0213 avatar Nov 05 '25 04:11 fengju0213

@xinrui-z hi,thanks for this pr! notice that #3338 has been merged ,Is this PR somewhat repetitive?

@fengju0213 hi, thanks for the feedback! Yes, I noticed #3338 was merged. Since then, we’ve made some further updates — mainly adding multi-client routing (now supporting multiple endpoints). I’ve also modified and extended the test cases to cover these changes.

xinrui-z avatar Nov 05 '25 06:11 xinrui-z