-
Notifications
You must be signed in to change notification settings - Fork 8
Milestone 1.8.0 #81
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
Milestone 1.8.0 #81
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements Milestone 1.8.0 with significant enhancements to the infrastructure and sample management system. The changes introduce automatic infrastructure detection and selection, streamlined notebook interfaces, and improved testing capabilities.
Key Changes
- Introduces infrastructure auto-detection and selection functionality through new helper classes
- Consolidates notebook structure with standardized environment initialization sections
- Adds comprehensive testing framework for infrastructure selection workflows
Reviewed Changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 5 comments.
Show a summary per file
File | Description |
---|---|
tests/python/test_utils.py | Extensive test coverage for infrastructure selection, query functionality, and deployment scenarios |
shared/python/utils.py | Core infrastructure management classes including InfrastructureNotebookHelper and enhanced NotebookHelper with auto-selection |
samples/*/create.ipynb | Standardized notebook structure with consistent environment initialization and API configuration sections |
infrastructure/*/create_infrastructure.py | New infrastructure creation modules providing programmatic deployment capabilities |
infrastructure/*/create.ipynb | Simplified infrastructure notebooks using new helper classes |
infrastructure/simple-apim/main.bicep | Removed deprecated revealBackendApiInfo parameter |
Resolves #80