Skip to content

Conversation

@rmitsch
Copy link
Collaborator

@rmitsch rmitsch commented Dec 21, 2025

Description

Adds logging of raw model responses when include_meta is True.

Related Issues

Changes Made

  • Adds logging of raw model responses when include_meta is True.
  • Aligns bridge typing with reality by changing typing of returns from Iterable to Sequence (iterative processing is handle by (core) task).

Checklist

  • Tests have been extended to cover changes in functionality
  • Existing and new tests succeed
  • Documentation updated (if applicable)
  • Related issues linked

Screenshots/Examples (if applicable)

@rmitsch rmitsch self-assigned this Dec 21, 2025
@codecov
Copy link

codecov bot commented Dec 21, 2025

Codecov Report

❌ Patch coverage is 96.00000% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
sieves/tasks/predictive/classification/bridges.py 84.21% 3 Missing ⚠️
sieves/model_wrappers/core.py 66.66% 1 Missing ⚠️
sieves/model_wrappers/dspy_.py 87.50% 1 Missing ⚠️
sieves/model_wrappers/outlines_.py 88.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #236      +/-   ##
==========================================
+ Coverage   93.29%   93.33%   +0.04%     
==========================================
  Files          62       62              
  Lines        3414     3450      +36     
==========================================
+ Hits         3185     3220      +35     
- Misses        229      230       +1     
Files with missing lines Coverage Δ
sieves/model_wrappers/gliner_.py 94.28% <100.00%> (+0.16%) ⬆️
sieves/model_wrappers/huggingface_.py 88.88% <100.00%> (+0.31%) ⬆️
sieves/model_wrappers/langchain_.py 87.50% <100.00%> (+0.32%) ⬆️
sieves/tasks/predictive/bridges.py 92.30% <100.00%> (+0.03%) ⬆️
sieves/tasks/predictive/core.py 91.16% <100.00%> (+0.40%) ⬆️
...tasks/predictive/information_extraction/bridges.py 96.61% <100.00%> (+0.03%) ⬆️
sieves/tasks/predictive/ner/bridges.py 93.92% <100.00%> (+0.06%) ⬆️
sieves/tasks/predictive/pii_masking/bridges.py 96.85% <100.00%> (+0.04%) ⬆️
...ves/tasks/predictive/question_answering/bridges.py 98.27% <100.00%> (+0.03%) ⬆️
...ves/tasks/predictive/sentiment_analysis/bridges.py 98.24% <100.00%> (+0.03%) ⬆️
... and 6 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rmitsch rmitsch marked this pull request as ready for review December 21, 2025 18:48
@rmitsch rmitsch merged commit eac19b8 into main Dec 21, 2025
3 checks passed
@rmitsch rmitsch deleted the 220 branch December 21, 2025 18:48
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.

Include raw model output in meta

1 participant