Skip to content

feat(data-collection): split generative-AI input/output collection#6679

Draft
ericapisani wants to merge 1 commit into
ep/db-spec-experiment-foundationfrom
ep/db-spec-experiment-genai
Draft

feat(data-collection): split generative-AI input/output collection#6679
ericapisani wants to merge 1 commit into
ep/db-spec-experiment-foundationfrom
ep/db-spec-experiment-genai

Conversation

@ericapisani

Copy link
Copy Markdown
Member

Part of the data_collection split — depends on #6676 (foundation).

Splits gen-AI content gates into input/output and changes include_prompts from bool=True to Optional[bool]=None (defer-to-global) across the AI integrations.

@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

89115 passed | ❌ 1 failed | ⏭️ 5890 skipped | Total: 95006 | Pass Rate: 93.8% | Execution Time: 315m 42s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -1450
Passed Tests 📉 -1036
Failed Tests 📈 +1
Skipped Tests 📉 -415

➕ New Tests (1)

View new tests
  • test_embeddings_create_with_list_input[False-False]
    • File: tests.integrations.litellm.test_litellm
    • Status: ❌ Failing

❌ Failed Tests

test_embeddings_create_with_list_input[False-False]

File: tests.integrations.litellm.test_litellm
Suite: py3.12-litellm-v1.77.7
Error: assert 0 == 1 + where 0 = len([])

Stack Trace
tests/integrations/litellm/test_litellm.py:1031: in test_embeddings_create_with_list_input
    assert len(spans) == 1
E   assert 0 == 1
E    +  where 0 = len([])

✅ Patch coverage is 96.63%. Project has 2419 uncovered lines.
✅ Project coverage is 89.93%. Comparing base (base) to head (head).

Files with missing lines (3)
File Patch % Lines
sentry_sdk/integrations/langchain.py 81.25% ⚠️ 3 Missing
sentry_sdk/integrations/google_genai/streaming.py 100.00% ⚠️ 1 partials
sentry_sdk/integrations/pydantic_ai/utils.py 100.00% ⚠️ 1 partials
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.92%    89.93%    +0.01%
==========================================
  Files          192       193        +1
  Lines        23809     24020      +211
  Branches      8218      8306       +88
==========================================
+ Hits         21409     21601      +192
- Misses        2400      2419       +19
- Partials      1339      1357       +18

Generated by Codecov Action

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.

1 participant