Skip to content

Update Radiology Insights SDK from local TypeSpec #41963

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

Closed

Conversation

catalinaperalta
Copy link
Member

Overview

This PR updates the Azure Health Insights Radiology Insights Python SDK based on the latest local TypeSpec specification.

Changes Made

New Features

  • Added �egin_infer_from_custom_model_id method to both sync and async clients for custom model inference
  • Added APIView properties for the new inference method

SDK Generation Updates

  • Updated SDK from local TypeSpec at: C:\Users\caperal\repos\azure-rest-api-specs\specification\ai\HealthInsights\HealthInsights.RadiologyInsights
  • Version bumped to 1.2.0b1 for beta release

Bug Fixes & Compatibility

  • Fixed import compatibility for collections.abc.MutableMapping with fallback to legacy import for older Python versions
  • Fixed classmethod call issue in model_base.py with appropriate pylint disable
  • Updated setup.py to use proper package namespace resolution
  • Fixed operation mixin class names and imports throughout the codebase

Static Validation

  • All changes pass pylint, mypy, pyright validation
  • Fixed import errors and type checking issues

Testing

  • All static validation steps completed successfully
  • SDK generation and update process completed without errors

Files Modified

  • Client and async client files
  • Operations and async operations files
  • Model base utilities
  • Setup and configuration files
  • APIView properties

This update ensures the SDK is current with the latest TypeSpec and maintains backward compatibility while adding new functionality.

…del inference method and fix compatibility issues

- Added begin_infer_from_custom_model_id method to both sync and async clients
- Fixed import compatibility for collections.abc.MutableMapping with fallback
- Fixed classmethod call issue in model_base.py with pylint disable
- Updated version to 1.2.0b1 for beta release
- Updated setup.py to use proper package namespace resolution
- Fixed operation mixin class names and imports
- Added APIView properties for new inference method
@catalinaperalta catalinaperalta force-pushed the feature/update-radiology-insights-typespec branch from 0aa7d91 to eed3b9d Compare July 9, 2025 22:40
Copy link

github-actions bot commented Jul 9, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

azure-healthinsights-radiologyinsights

- Updated method name from begin_infer_from_custom_model_id to begin_custom_inference
- Updated model name from RadiologyInsightsCustomInferenceResponse to RadiologyInsightsCustomInferenceResult
- Fixed collections.abc import compatibility for Python 3.8+
- Fixed pylint issues with classmethod call in model_base.py
- Updated patch files with correct method signatures and model references
- Updated API view properties with correct mappings
@catalinaperalta catalinaperalta deleted the feature/update-radiology-insights-typespec branch July 15, 2025 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants