Skip to content

Conversation

@rmitsch
Copy link
Collaborator

@rmitsch rmitsch commented Nov 16, 2025

Description

Removes all reasoning fields in task signatures. We consider this superfluous given today's generative models inbuilt steerable reasoning levels.

Related Issues

Changes Made

  • Removes reasoning fields in code and docs.

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 Nov 16, 2025
@codecov
Copy link

codecov bot commented Nov 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #205      +/-   ##
==========================================
- Coverage   92.63%   92.51%   -0.13%     
==========================================
  Files          63       63              
  Lines        3339     3298      -41     
==========================================
- Hits         3093     3051      -42     
- Misses        246      247       +1     
Files with missing lines Coverage Δ
sieves/tasks/predictive/classification/bridges.py 88.18% <100.00%> (+0.11%) ⬆️
sieves/tasks/predictive/classification/core.py 94.76% <ø> (-0.07%) ⬇️
sieves/tasks/predictive/core.py 90.58% <ø> (-0.06%) ⬇️
...tasks/predictive/information_extraction/bridges.py 97.56% <100.00%> (-0.14%) ⬇️
...es/tasks/predictive/information_extraction/core.py 92.04% <ø> (-0.09%) ⬇️
sieves/tasks/predictive/ner/bridges.py 94.78% <ø> (-0.05%) ⬇️
sieves/tasks/predictive/pii_masking/bridges.py 98.54% <100.00%> (-0.08%) ⬇️
sieves/tasks/predictive/pii_masking/core.py 94.44% <ø> (-0.08%) ⬇️
...ves/tasks/predictive/question_answering/bridges.py 99.12% <100.00%> (-0.03%) ⬇️
sieves/tasks/predictive/question_answering/core.py 95.38% <ø> (-0.07%) ⬇️
... and 4 more

... and 1 file with indirect coverage changes

🚀 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 November 16, 2025 15:06
@rmitsch rmitsch merged commit 12a4ac7 into main Nov 16, 2025
3 checks passed
@rmitsch rmitsch deleted the feat/204-remove-reasoning-fields branch November 16, 2025 15:06
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.

Remove reasoning fields + processing

1 participant