Bugfix - MAG Video Indexer API & Tweak to Speech Service Defaults #344
Conversation
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
these are pending release and will show up in next version.
…nt cloud and is not read-only in the admin settings, removed hardcoded commercial speech_service_endpoint and speech_service_location
…t format based on location and cloud type
|
@microsoft-github-policy-service agree |
|
Hello, I appreciate taking the time to improve Simple Chat! Ive update the merge branch to Development. I will resolve the conflicts and review your changes. I will keep you in the loop via comments. |
…t end/blob/cosmos. Includes adding the file's video indexer ID to the file metadata in cosmos.
I merged in all the current changes in Development and also added the process to delete videos from the Video Indexer service when they're deleted from the workspace. Currently they're only deleted from blob storage and cosmos but are not deleted from the VI service. |
paullizer
added a commit
that referenced
this pull request
Sep 3, 2025
* Documentation: Refactor (#406) * Add new images and documentation for Simple Chat setup and infrastructure - Added various images for UI and functionality demonstrations. - Created 'infrastructure_as_code.md' for infrastructure setup guidance. - Added 'setup_instructions.md' detailing deployment options and instructions. - Introduced 'setup_instructions_special.md' for special deployment scenarios including Azure Government configurations and Managed Identity usage. - Updated documentation to enhance clarity and usability for developers. * Update README setup instructions and remove obsolete infrastructure documentation * Fix TOC anchors: README and application_workflows.md --------- Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> * Improvements to Terraform deployment process (#400) * Delete logfile.log * Removed external these are pending release and will show up in next version. * Update route_external_health.py * Adding health check * Added health check feature * updating app.py, removed external * Updated to v0.215.36 * Update app.py * v0.215.37 * v0.215.37 * Update app.py (#359) * Update app.py Super tactical fix to allow ChainGuard container to start * updated to v0.215.38 --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> * Update README.md * Refactor devcontainer.json to update feature sources and enhance VS Code customizations * Added powershell script to configure Base dependencies. * remove tfvars files from solution * fix: set authentication requirements for app * fix: redirect to correct auth provider * fix: add alternate auth endpoint * fix: corrected image name in app * fix: use managed identity for image retrevial * fix: upate app docker registry server to dynamic config * Documentation updates for deployment with Terraform * removed unused variable from powershell script. * fix: enable shared access key for Azure Storage Account --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> Co-authored-by: Paul Lizer <paul.lizer@live.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> * Bugfix/govcloud document intelligence, ai search, content safety managed identity authentication (#388) * Add support for govt and custom search resource manager, update document intelligence, ai search, and content safety client initialization for govt and custom environments * Fix Azure Document Intelligence operation via managed identity authentication for government and custom environments by adding base64 encoding support for document uploads, which is required by current GA document intelligence API version (2024-11-30) * Fix Azure AI services (content safety, document intelligence, search) managed identity integration for government and custom environments. Consolidated Document Intelligence calls to use same module and API and added base64 document uploads in Document Intelligence API, required by most current API GA version (2024-11-30). * Update README.md to correct Managed Identity role requirements for Azure services * Add support for govt and custom search resource manager, update document intelligence, ai search, and content safety client initialization for govt and custom environments * Fix Azure Document Intelligence operation via managed identity authentication for government and custom environments by adding base64 encoding support for document uploads, which is required by current GA document intelligence API version (2024-11-30) * Fix Azure AI services (content safety, document intelligence, search) managed identity integration for government and custom environments. Consolidated Document Intelligence calls to use same module and API and added base64 document uploads in Document Intelligence API, required by most current API GA version (2024-11-30). * Update README.md to correct Managed Identity role requirements for Azure services * added search_client_public to managed identity auth flow --------- Co-authored-by: Joshua Wilshere <joshua.wilshere@oig.dhs.gov> Co-authored-by: Paul Lizer <paullizer@microsoft.com> * Feature/code highlighting (#363) * Delete logfile.log * Removed external these are pending release and will show up in next version. * Update route_external_health.py * Adding health check * Added health check feature * updating app.py, removed external * Updated to v0.215.36 * Update app.py * v0.215.37 * v0.215.37 * Update app.py (#359) * Update app.py Super tactical fix to allow ChainGuard container to start * updated to v0.215.38 --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> * Update README.md * Feature - added code highlighting in code blocks returned by the GPT model using Prism (www.prismjs.com) with themes for light and dark mode. Also made adjustments to <pre> and <pre><code> css to try and prevent double scrolls bars when code blocks overflow available space in chat box. --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> Co-authored-by: Paul Lizer <paul.lizer@live.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> * Bugfix - MAG Video Indexer API & Tweak to Speech Service Defaults (#344) * Delete logfile.log * Removed external these are pending release and will show up in next version. * Update route_external_health.py * Adding health check * Added health check feature * updating app.py, removed external * Updated to v0.215.36 * Update app.py * v0.215.37 * v0.215.37 * updated so video indexer api default changes based on public/government cloud and is not read-only in the admin settings, removed hardcoded commercial speech_service_endpoint and speech_service_location * added placeholder for speech_service_endpoint to show default endpoint format based on location and cloud type * added reminders to save pending changes before fetching GPT models in admin settings * Added process to delete videos from VI service when deleted from front end/blob/cosmos. Includes adding the file's video indexer ID to the file metadata in cosmos. * Added unmerged files from Development branch --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> Co-authored-by: Paul Lizer <paul.lizer@live.com> --------- Co-authored-by: SteveCInVA <37545884+SteveCInVA@users.noreply.github.com> Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> Co-authored-by: Xeelee33 <Xeelee33@users.noreply.github.com> Co-authored-by: Joshua Wilshere <joshua.wilshere@oig.dhs.gov>
paullizer
added a commit
that referenced
this pull request
Sep 4, 2025
* Development (#409) * Documentation: Refactor (#406) * Add new images and documentation for Simple Chat setup and infrastructure - Added various images for UI and functionality demonstrations. - Created 'infrastructure_as_code.md' for infrastructure setup guidance. - Added 'setup_instructions.md' detailing deployment options and instructions. - Introduced 'setup_instructions_special.md' for special deployment scenarios including Azure Government configurations and Managed Identity usage. - Updated documentation to enhance clarity and usability for developers. * Update README setup instructions and remove obsolete infrastructure documentation * Fix TOC anchors: README and application_workflows.md --------- Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> * Improvements to Terraform deployment process (#400) * Delete logfile.log * Removed external these are pending release and will show up in next version. * Update route_external_health.py * Adding health check * Added health check feature * updating app.py, removed external * Updated to v0.215.36 * Update app.py * v0.215.37 * v0.215.37 * Update app.py (#359) * Update app.py Super tactical fix to allow ChainGuard container to start * updated to v0.215.38 --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> * Update README.md * Refactor devcontainer.json to update feature sources and enhance VS Code customizations * Added powershell script to configure Base dependencies. * remove tfvars files from solution * fix: set authentication requirements for app * fix: redirect to correct auth provider * fix: add alternate auth endpoint * fix: corrected image name in app * fix: use managed identity for image retrevial * fix: upate app docker registry server to dynamic config * Documentation updates for deployment with Terraform * removed unused variable from powershell script. * fix: enable shared access key for Azure Storage Account --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> Co-authored-by: Paul Lizer <paul.lizer@live.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> * Bugfix/govcloud document intelligence, ai search, content safety managed identity authentication (#388) * Add support for govt and custom search resource manager, update document intelligence, ai search, and content safety client initialization for govt and custom environments * Fix Azure Document Intelligence operation via managed identity authentication for government and custom environments by adding base64 encoding support for document uploads, which is required by current GA document intelligence API version (2024-11-30) * Fix Azure AI services (content safety, document intelligence, search) managed identity integration for government and custom environments. Consolidated Document Intelligence calls to use same module and API and added base64 document uploads in Document Intelligence API, required by most current API GA version (2024-11-30). * Update README.md to correct Managed Identity role requirements for Azure services * Add support for govt and custom search resource manager, update document intelligence, ai search, and content safety client initialization for govt and custom environments * Fix Azure Document Intelligence operation via managed identity authentication for government and custom environments by adding base64 encoding support for document uploads, which is required by current GA document intelligence API version (2024-11-30) * Fix Azure AI services (content safety, document intelligence, search) managed identity integration for government and custom environments. Consolidated Document Intelligence calls to use same module and API and added base64 document uploads in Document Intelligence API, required by most current API GA version (2024-11-30). * Update README.md to correct Managed Identity role requirements for Azure services * added search_client_public to managed identity auth flow --------- Co-authored-by: Joshua Wilshere <joshua.wilshere@oig.dhs.gov> Co-authored-by: Paul Lizer <paullizer@microsoft.com> * Feature/code highlighting (#363) * Delete logfile.log * Removed external these are pending release and will show up in next version. * Update route_external_health.py * Adding health check * Added health check feature * updating app.py, removed external * Updated to v0.215.36 * Update app.py * v0.215.37 * v0.215.37 * Update app.py (#359) * Update app.py Super tactical fix to allow ChainGuard container to start * updated to v0.215.38 --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> * Update README.md * Feature - added code highlighting in code blocks returned by the GPT model using Prism (www.prismjs.com) with themes for light and dark mode. Also made adjustments to <pre> and <pre><code> css to try and prevent double scrolls bars when code blocks overflow available space in chat box. --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> Co-authored-by: Paul Lizer <paul.lizer@live.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> * Bugfix - MAG Video Indexer API & Tweak to Speech Service Defaults (#344) * Delete logfile.log * Removed external these are pending release and will show up in next version. * Update route_external_health.py * Adding health check * Added health check feature * updating app.py, removed external * Updated to v0.215.36 * Update app.py * v0.215.37 * v0.215.37 * updated so video indexer api default changes based on public/government cloud and is not read-only in the admin settings, removed hardcoded commercial speech_service_endpoint and speech_service_location * added placeholder for speech_service_endpoint to show default endpoint format based on location and cloud type * added reminders to save pending changes before fetching GPT models in admin settings * Added process to delete videos from VI service when deleted from front end/blob/cosmos. Includes adding the file's video indexer ID to the file metadata in cosmos. * Added unmerged files from Development branch --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> Co-authored-by: Paul Lizer <paul.lizer@live.com> --------- Co-authored-by: SteveCInVA <37545884+SteveCInVA@users.noreply.github.com> Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> Co-authored-by: Xeelee33 <Xeelee33@users.noreply.github.com> Co-authored-by: Joshua Wilshere <joshua.wilshere@oig.dhs.gov> * Improved New Action workflow * fixed global agent / action loading bug --------- Co-authored-by: SteveCInVA <37545884+SteveCInVA@users.noreply.github.com> Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> Co-authored-by: Xeelee33 <Xeelee33@users.noreply.github.com> Co-authored-by: Joshua Wilshere <joshua.wilshere@oig.dhs.gov>
paullizer
added a commit
that referenced
this pull request
Sep 5, 2025
* Development (#409) * Documentation: Refactor (#406) * Add new images and documentation for Simple Chat setup and infrastructure - Added various images for UI and functionality demonstrations. - Created 'infrastructure_as_code.md' for infrastructure setup guidance. - Added 'setup_instructions.md' detailing deployment options and instructions. - Introduced 'setup_instructions_special.md' for special deployment scenarios including Azure Government configurations and Managed Identity usage. - Updated documentation to enhance clarity and usability for developers. * Update README setup instructions and remove obsolete infrastructure documentation * Fix TOC anchors: README and application_workflows.md --------- Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> * Improvements to Terraform deployment process (#400) * Delete logfile.log * Removed external these are pending release and will show up in next version. * Update route_external_health.py * Adding health check * Added health check feature * updating app.py, removed external * Updated to v0.215.36 * Update app.py * v0.215.37 * v0.215.37 * Update app.py (#359) * Update app.py Super tactical fix to allow ChainGuard container to start * updated to v0.215.38 --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> * Update README.md * Refactor devcontainer.json to update feature sources and enhance VS Code customizations * Added powershell script to configure Base dependencies. * remove tfvars files from solution * fix: set authentication requirements for app * fix: redirect to correct auth provider * fix: add alternate auth endpoint * fix: corrected image name in app * fix: use managed identity for image retrevial * fix: upate app docker registry server to dynamic config * Documentation updates for deployment with Terraform * removed unused variable from powershell script. * fix: enable shared access key for Azure Storage Account --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> Co-authored-by: Paul Lizer <paul.lizer@live.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> * Bugfix/govcloud document intelligence, ai search, content safety managed identity authentication (#388) * Add support for govt and custom search resource manager, update document intelligence, ai search, and content safety client initialization for govt and custom environments * Fix Azure Document Intelligence operation via managed identity authentication for government and custom environments by adding base64 encoding support for document uploads, which is required by current GA document intelligence API version (2024-11-30) * Fix Azure AI services (content safety, document intelligence, search) managed identity integration for government and custom environments. Consolidated Document Intelligence calls to use same module and API and added base64 document uploads in Document Intelligence API, required by most current API GA version (2024-11-30). * Update README.md to correct Managed Identity role requirements for Azure services * Add support for govt and custom search resource manager, update document intelligence, ai search, and content safety client initialization for govt and custom environments * Fix Azure Document Intelligence operation via managed identity authentication for government and custom environments by adding base64 encoding support for document uploads, which is required by current GA document intelligence API version (2024-11-30) * Fix Azure AI services (content safety, document intelligence, search) managed identity integration for government and custom environments. Consolidated Document Intelligence calls to use same module and API and added base64 document uploads in Document Intelligence API, required by most current API GA version (2024-11-30). * Update README.md to correct Managed Identity role requirements for Azure services * added search_client_public to managed identity auth flow --------- Co-authored-by: Joshua Wilshere <joshua.wilshere@oig.dhs.gov> Co-authored-by: Paul Lizer <paullizer@microsoft.com> * Feature/code highlighting (#363) * Delete logfile.log * Removed external these are pending release and will show up in next version. * Update route_external_health.py * Adding health check * Added health check feature * updating app.py, removed external * Updated to v0.215.36 * Update app.py * v0.215.37 * v0.215.37 * Update app.py (#359) * Update app.py Super tactical fix to allow ChainGuard container to start * updated to v0.215.38 --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> * Update README.md * Feature - added code highlighting in code blocks returned by the GPT model using Prism (www.prismjs.com) with themes for light and dark mode. Also made adjustments to <pre> and <pre><code> css to try and prevent double scrolls bars when code blocks overflow available space in chat box. --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> Co-authored-by: Paul Lizer <paul.lizer@live.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> * Bugfix - MAG Video Indexer API & Tweak to Speech Service Defaults (#344) * Delete logfile.log * Removed external these are pending release and will show up in next version. * Update route_external_health.py * Adding health check * Added health check feature * updating app.py, removed external * Updated to v0.215.36 * Update app.py * v0.215.37 * v0.215.37 * updated so video indexer api default changes based on public/government cloud and is not read-only in the admin settings, removed hardcoded commercial speech_service_endpoint and speech_service_location * added placeholder for speech_service_endpoint to show default endpoint format based on location and cloud type * added reminders to save pending changes before fetching GPT models in admin settings * Added process to delete videos from VI service when deleted from front end/blob/cosmos. Includes adding the file's video indexer ID to the file metadata in cosmos. * Added unmerged files from Development branch --------- Co-authored-by: Paul Lizer <paullizer@microsoft.com> Co-authored-by: Paul Lizer <paul.lizer@live.com> --------- Co-authored-by: SteveCInVA <37545884+SteveCInVA@users.noreply.github.com> Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> Co-authored-by: Xeelee33 <Xeelee33@users.noreply.github.com> Co-authored-by: Joshua Wilshere <joshua.wilshere@oig.dhs.gov> * Improved New Action workflow * fixed global agent / action loading bug * Fixed upload bug * Fixed file upload bug and tabular data in chat * Adding fix markdowns, functional tests, bug fix for tabular data in convos --------- Co-authored-by: SteveCInVA <37545884+SteveCInVA@users.noreply.github.com> Co-authored-by: Steve Carroll <Stephen.Carroll@microsoft.com> Co-authored-by: Aaron Barth <aaronba@users.noreply.github.com> Co-authored-by: Xeelee33 <Xeelee33@users.noreply.github.com> Co-authored-by: Joshua Wilshere <joshua.wilshere@oig.dhs.gov>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Added logic to change the video indexer api based on commercial or government cloud, and also added ability for admin to change/update the default values in the admin settings. Originally the endpoint and api version have "readonly" constraints in the form control and default to the commercial endpoint, so it's not possible to change the endpoint or API version in the admin settings UI.
Added process to delete video from the Video Indexer service when a user deletes it from the workspace. This required adding the file's Video Indexer ID to the metadata in Cosmos DB.
Removed the hardcoded speech service endpoint and location (currently defaulting to commercial/eastus) and instead added a placeholder to show endpoint format within the field.
Added text reminders to the GPT admin settings tabs to save pending settings before fetching models.
Speech service endpoint placeholder
GPT Save pending changes reminder text
Embeddings Save pending changes reminder text
