Open
Conversation
- Updated the content_extraction.ipynb notebook to use the new Azure AI Content Understanding SDK. - Replaced deprecated methods and adjusted the code for asynchronous operations. - Improved the structure of the notebook for better readability and organization. - Added a new sample_helper.py file containing utility functions for handling analysis results, saving images, and extracting operation IDs. - Enhanced error handling and logging throughout the notebook.
- Updated `.gitignore` to exclude `test_output/` directory. - Added new face images for enrollment and testing. - Refactored `build_person_directory.ipynb` to use async methods and improved logging. - Updated person and face management logic to handle Azure SDK changes. - Improved error handling and logging for face and person operations. - Enhanced `content_extraction.ipynb` with audio analysis capabilities and cleanup logic.
- Updated `analyzer_training.ipynb` to enhance client initialization and error handling. - Modified training data path handling and SAS URL generation for better clarity. - Improved analyzer creation process with unique ID generation and logging. - Enhanced document analysis with operation ID extraction and result retrieval. - Updated `build_person_directory.ipynb` to streamline face addition and identification processes. - Refined face association and disassociation logic for better clarity and functionality. - Improved person directory updates with clearer resource handling. - Updated `classifier.ipynb` to enhance classifier ID generation and result handling. - Improved operation ID extraction and result retrieval for classification tasks. - Refactored `content_extraction.ipynb` to standardize analyzer ID usage and improve file handling. - Enhanced audio and video analysis processes with clearer logging and data handling. - Updated kernel specifications across notebooks for consistency.
changjian-wang
had a problem deploying
to
MMI-Samples
GitHub Actions
Failure
| "Before creating the analyzer, fill in the constant `ANALYZER_ID` with a relevant name for your task. In this example, we generate a unique suffix so that this cell can be run multiple times to create different analyzers.\n", | ||
| "\n", | ||
| "We use **training_data_sas_url** and **training_data_path** as set in the [.env](./.env) file and used in the previous step." | ||
| "We use **TRAINING_DATA_SAS_URL** and **TRAINING_DATA_PATH** as set in the [.env](./.env) file and used in the previous step." |
Member
There was a problem hiding this comment.
@changjian-wang I saw your new commits. Sorry for the confusion, I was referring to this specific line. I mean we're using lower-case training_data_sas_url and training_data_path as variables within the notebook. I think we may not need to mention .env file here like "We use training_data_sas_url and training_data_path set in the previous step."
| "from azure.storage.blob import ContainerSasPermissions\n", | ||
| "from azure.core.credentials import AzureKeyCredential\n", | ||
| "from azure.identity import DefaultAzureCredential\n", | ||
| "from azure.ai.contentunderstanding.aio import ContentUnderstandingClient\n", |
Member
There was a problem hiding this comment.
We had some module errors in auto check. We will need to add it into requirements.txt.
changjian-wang
had a problem deploying
to
MMI-Samples
GitHub Actions
Failure
changjian-wang
had a problem deploying
to
MMI-Samples
GitHub Actions
Failure
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Update samples from sdk
Does this introduce a breaking change?
Pull Request Type
What kind of change does this Pull Request introduce?