Skip to content

feat: Creating custom input and output types for multimodal ingestion#500

Open
pratduv wants to merge 1 commit intomainfrom
sc-58230/local-ingest-models
Open

feat: Creating custom input and output types for multimodal ingestion#500
pratduv wants to merge 1 commit intomainfrom
sc-58230/local-ingest-models

Conversation

@pratduv
Copy link
Contributor

@pratduv pratduv commented Mar 12, 2026

User description

Shortcut:

Description:

Tests:

  • Unit Tests Added
  • E2E Test Added (if it's a user-facing feature, or fixing a bug)

Generated description

Below is a concise technical summary of the changes proposed in this PR:
Enable multimodal ingestion by widening SDK-local trace/span models (e.g., LoggedTrace, LoggedLlmSpan, LoggedAgentSpan) to accept LoggedMessages containing IngestContentBlocks and document the richer ingest payload types. Update the tracing builders/loggers (TraceBuilder, GalileoLogger) so they emit the new logged models, keep metrics, and ensure ingestion helpers/tests target the logged payloads for Consistency with IngestContentBlocks.

TopicDetails
Multimodal schema Expand LoggedTrace/span models, LoggedMessage, and the ingest payloads to accept the new IngestContentBlock/IngestMessageContent types so multimodal content can be submitted without coercion.
Modified files (6)
  • src/galileo/schema/__init__.py
  • src/galileo/schema/content_blocks.py
  • src/galileo/schema/logged.py
  • src/galileo/schema/message.py
  • src/galileo/schema/trace.py
  • tests/schemas/test_logged.py
Latest Contributors(1)
UserCommitDate
setu@rungalileo.iochore-Setup-repo-packa...February 22, 2025
Trace logging Adapt TraceBuilder/GalileoLogger (and related batch tests) to build, stream, and ingest the widened LoggedTrace/span hierarchy, preserving metrics and parent relationships.
Modified files (3)
  • galileo-adk/src/galileo_adk/trace_builder.py
  • src/galileo/logger/logger.py
  • tests/test_logger_batch.py
Latest Contributors(2)
UserCommitDate
calebe.sep@hotmail.comfix-auto-convert-non-s...March 02, 2026
sankar@galileo.aifeat-galileo-adk-to-na...February 22, 2026
This pull request is reviewed by Baz. Review like a pro on (Baz).

@pratduv pratduv requested a review from a team as a code owner March 12, 2026 21:58
@pratduv pratduv requested review from jonahe and removed request for jonahe March 12, 2026 21:58
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from 898ea7b to 8765922 Compare March 12, 2026 22:33
@codecov
Copy link

codecov bot commented Mar 12, 2026

Codecov Report

❌ Patch coverage is 98.18182% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.00%. Comparing base (66e86cd) to head (f2c95c6).

Files with missing lines Patch % Lines
src/galileo/logger/logger.py 95.00% 2 Missing ⚠️
galileo-adk/src/galileo_adk/trace_builder.py 96.29% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #500      +/-   ##
==========================================
+ Coverage   82.13%   83.00%   +0.86%     
==========================================
  Files          96      109      +13     
  Lines        9282    10514    +1232     
==========================================
+ Hits         7624     8727    +1103     
- Misses       1658     1787     +129     
Flag Coverage Δ
galileo-adk 88.70% <96.29%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from 8765922 to 11330cb Compare March 12, 2026 23:11
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from 11330cb to 7ca3129 Compare March 13, 2026 01:34
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from 7ca3129 to a7bb086 Compare March 13, 2026 01:52
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from a7bb086 to 1a29b4f Compare March 13, 2026 02:31
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from 1a29b4f to 2405b0e Compare March 13, 2026 04:12
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from 2405b0e to de35999 Compare March 13, 2026 15:33
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from de35999 to 85fd9f5 Compare March 13, 2026 19:38
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch 2 times, most recently from ce28531 to 1875ca6 Compare March 13, 2026 20:20
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from 1875ca6 to c9eb90e Compare March 13, 2026 23:45
@pratduv pratduv force-pushed the sc-58230/local-ingest-models branch from c9eb90e to f2c95c6 Compare March 14, 2026 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant