Skip to content

Conversation

@ArneBinder
Copy link
Owner

@ArneBinder ArneBinder commented Aug 5, 2025

This PR allows it to easily overwrite that functionality in downstream projects.

some context/motivation: ArneBinder/pie-datasets#200.

@codecov
Copy link

codecov bot commented Aug 5, 2025

Codecov Report

❌ Patch coverage is 22.22222% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.53%. Comparing base (f99c96f) to head (5df85e1).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/pytorch_ie/datamodule.py 22.22% 7 Missing ⚠️

❌ Your patch status has failed because the patch coverage (22.22%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #490      +/-   ##
==========================================
- Coverage   78.34%   72.53%   -5.81%     
==========================================
  Files          32       32              
  Lines        2161     2163       +2     
  Branches      316      316              
==========================================
- Hits         1693     1569     -124     
- Misses        377      513     +136     
+ Partials       91       81      -10     

☔ 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.

@ArneBinder ArneBinder merged commit 445e457 into main Aug 5, 2025
3 of 4 checks passed
@ArneBinder ArneBinder deleted the datamodule/outsource_encode_documents branch August 5, 2025 01:25
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.

2 participants