Replace Azure Inference URL with GitHub Models API URL#236
Conversation
…el names Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
|
@Copilot Actually, the GITHUB_BASE_URL env var isnt needed, since it is always the same. Can you remove that and always use "https://models.github.ai/inference"? |
Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
Removed the GITHUB_BASE_URL environment variable and hard-coded the GitHub Models URL as requested. The URL |
|
@copilot The test file needs formatting with ruff, see the failed CI. |
Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
Fixed the formatting issues in the test file using ruff. All formatting and linting checks now pass. (967a13a) |
This PR migrates the codebase from the deprecated Azure inference endpoint to the new GitHub Models API endpoint, as announced in the GitHub Models deprecation notice.
Changes Made
🔗 URL Migration
https://models.inference.ai.azure.comwithhttps://models.github.ai/inferencesrc/backend/fastapi_app/openai_clients.py(2 instances).env.sample🏷️ Model Name Updates
gpt-4o→openai/gpt-4otext-embedding-3-large→openai/text-embedding-3-largeopenai_clients.pyanddependencies.py.env.sample🧪 Test Coverage
Example Usage
After this change, the correct configuration for GitHub Models is:
Validation
Fixes #235.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.