Skip to content

Conversation

@rads-1996
Copy link
Member

Image Spec POC

Copilot AI review requested due to automatic review settings November 3, 2025 17:19
@github-actions github-actions bot changed the title Basic I/O for Encoded Image Python: Basic I/O for Encoded Image Nov 3, 2025
Copy link
Contributor

Copilot AI left a 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 adds a new Python sample demonstrating image analysis with Azure AI observability integration. The sample shows how to load an image, encode it as base64, send it for analysis using Azure OpenAI, and collect telemetry via Application Insights.

  • Integrates image processing with Azure AI observability features
  • Demonstrates base64 encoding of images for multimodal input
  • Shows tracing setup for monitoring AI agent interactions
Comments suppressed due to low confidence (8)

python/samples/getting_started/image_spec_poc/basic_input_output_with_observability.py:6

  • Import of 'json' is not used.
import json

python/samples/getting_started/image_spec_poc/basic_input_output_with_observability.py:8

  • Import of 'datetime' is not used.
    Import of 'timezone' is not used.
from datetime import datetime, timezone

python/samples/getting_started/image_spec_poc/basic_input_output_with_observability.py:9

  • Import of 'Any' is not used.
from typing import Any

python/samples/getting_started/image_spec_poc/basic_input_output_with_observability.py:10

  • Import of 'Annotated' is not used.
from typing import Annotated

python/samples/getting_started/image_spec_poc/basic_input_output_with_observability.py:11

  • Import of 'BytesIO' is not used.
from io import BytesIO

python/samples/getting_started/image_spec_poc/basic_input_output_with_observability.py:14

  • Import of 'ChatAgent' is not used.
from agent_framework import ChatMessage, DataContent, Role, TextContent, ChatAgent

python/samples/getting_started/image_spec_poc/basic_input_output_with_observability.py:16

  • Import of 'Field' is not used.
from pydantic import Field

python/samples/getting_started/image_spec_poc/basic_input_output_with_observability.py:23

  • Import of 'SQLiteImageStore' is not used.
from db_setup import SQLiteImageStore

@rads-1996 rads-1996 marked this pull request as draft November 3, 2025 17:27
@rads-1996
Copy link
Member Author

Can refer to trace id - trace_id: a49ce82b63e0653ea72ae5a397f2f051

Also, the path for the image might vary

Copy link
Contributor

@moonbox3 moonbox3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We appreciate the contribution. What exact scenario are we trying to solve with these samples? Do we have a gap?

rads-1996 and others added 11 commits December 18, 2025 07:44
…ut_with_observability.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ut_with_observability.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ut_with_observability.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ut_with_observability.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ut_with_observability.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ut_with_observability.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@rads-1996
Copy link
Member Author

We appreciate the contribution. What exact scenario are we trying to solve with these samples? Do we have a gap?

This is a POC for an image specification for semantic conventions which will ultimately be raised in upstream opentelemetry. This is not a gap but something we are working on to present in upstream OTEL. Sorry for the confusion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants