Skip to content

Multiple changes from insiders #128

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Multiple changes from insiders #128

wants to merge 13 commits into from

Conversation

placerda
Copy link
Collaborator

This pull request includes various changes across multiple files to enhance Azure integration, improve error handling, and refactor code for better maintainability. The most important changes include adding Azure Monitor tracing, updating the get_answer function to accept credentials, and refactoring the create_kernel function to support different deployment scenarios.

Azure Integration Enhancements:

  • orc/code_orchestration.py: Added imports for Azure Monitor and AI Inference, and enabled Azure Monitor tracing if the APPLICATION_INSIGHTS_CONNECTION_STRING environment variable is set. Updated get_answer function to accept chainedTokenCredential and initialize the kernel with it. [1] [2] [3]

Error Handling Improvements:

  • orc/code_orchestration.py: Wrapped the security hub audit invocation in a try-except block to log errors if the audit fails.

Function and Variable Updates:

Code Refactoring:

  • shared/util.py: Refactored create_kernel to support different deployment scenarios and added a DummyAsyncContextManager class for AI Foundry deployments. Updated get_aoai_config to include model_endpoint in the configuration. [1] [2] [3] [4]

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.

3 participants