Skip to content

Conversation

@tmsjngx0
Copy link
Contributor

@tmsjngx0 tmsjngx0 commented Apr 2, 2025

Context

I'm adding support for Azure AI Inference Service with DeepSeek-V3 model integration. This enhancement allows users to utilize Azure AI's infrastructure to run the DeepSeek-V3 model, providing a more robust and scalable solution for AI inference. The implementation ensures proper handling of Azure-specific API paths, model parameters, and response formats.

Implementation

The implementation includes several key components:

  1. Azure AI Inference Service Integration:

    • Added model parameter to Azure AI Inference Service client configuration
    • Implemented dedicated handler for Azure AI Inference Service models
    • Added proper path handling for Azure AI endpoints
  2. DeepSeek-V3 Model Support:

    • Added support for DeepSeek-V3 model in Azure AI
    • Implemented proper message formatting for Azure AI
    • Added temperature settings specific to DeepSeek-V3
  3. Testing Infrastructure:

    • Added comprehensive tests for Azure AI Inference Service
    • Implemented tests for both streaming and non-streaming responses
    • Added tests for completePrompt functionality

Key technical decisions:

  • Used Azure AI's standard API paths for model inference
  • Implemented proper error handling for Azure-specific responses
  • Maintained backward compatibility with existing OpenAI integration

Screenshots

before after
No Azure AI Inference Service support Azure AI Inference Service with DeepSeek-V3 model support

How to Test

  1. Set up Azure AI service:

    • Configure your Azure AI service URL
    • Set up your API key
    • Select DeepSeek-V3 model ID
  2. Test Azure AI Integration:

    • Send a test prompt to verify Azure AI connection
    • Check both streaming and non-streaming responses
    • Verify model parameters are correctly passed
  3. Verify Error Handling:

    • Test with invalid API key
    • Test with incorrect service URL
    • Verify proper error messages
  4. Run Tests:

    npm test

    All Azure AI Inference Service tests should pass

Questions?

If you have any questions about the implementation or need clarification on any part of the changes, please feel free to ask in the PR comments.


Important

Add support for Azure AI Inference Service with DeepSeek-V3 model in OpenAiHandler, including Azure-specific path handling and comprehensive tests.

  • Azure AI Inference Service Integration:
    • Added support for Azure AI Inference Service in OpenAiHandler in openai.ts.
    • Implemented Azure-specific path handling with AZURE_AI_INFERENCE_PATH.
    • Added isAzure flag to determine Azure usage.
  • DeepSeek-V3 Model Support:
    • Integrated DeepSeek-V3 model with Azure AI in openai.ts.
    • Added temperature settings specific to DeepSeek-V3.
  • Testing:
    • Added tests for Azure AI Inference Service in openai.test.ts.
    • Implemented tests for streaming and non-streaming responses.
    • Verified API calls with correct Azure paths in requesty.test.ts.

This description was created by Ellipsis for bc0f90c. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Apr 2, 2025

⚠️ No Changeset found

Latest commit: 9ff9b54

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels Apr 2, 2025
@tmsjngx0 tmsjngx0 marked this pull request as draft April 2, 2025 22:55
@tmsjngx0 tmsjngx0 marked this pull request as ready for review April 2, 2025 23:46
@tmsjngx0 tmsjngx0 marked this pull request as draft April 2, 2025 23:55
@tmsjngx0 tmsjngx0 marked this pull request as ready for review April 3, 2025 00:00
Copy link
Collaborator

@mrubens mrubens left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 3, 2025
@mrubens mrubens merged commit e9bcee5 into RooCodeInc:main Apr 3, 2025
12 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Apr 3, 2025
@tmsjngx0 tmsjngx0 deleted the feature/azure-ai-inference-service branch April 3, 2025 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants