Skip to content

Add small fixes to improve handling of batch processing#46

Merged
lauraporta merged 1 commit intomainfrom
fixes-on-batch-processing
May 19, 2025
Merged

Add small fixes to improve handling of batch processing#46
lauraporta merged 1 commit intomainfrom
fixes-on-batch-processing

Conversation

@lauraporta
Copy link
Copy Markdown
Member

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other

Why is this PR needed?
Small fixes coming from errors that emerged during batch processing.

What does this PR do?
Small names changes, paths changes and removal of redundant method.

How has this PR been tested?

Tested locally and on CIs

Is this a breaking change?

Nope

Does this PR require an update to the documentation?

Yes

Checklist:

  • The code has been tested locally
  • Tests have been added to cover all new functionality
  • The documentation has been updated to reflect any changes
  • The code has been formatted with pre-commit

@codecov
Copy link
Copy Markdown

codecov bot commented May 16, 2025

Codecov Report

Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 44.12%. Comparing base (9b3b246) to head (993b733).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...tation/analysis/incremental_derotation_pipeline.py 0.00% 3 Missing ⚠️
derotation/derotate_batch.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
+ Coverage   43.95%   44.12%   +0.17%     
==========================================
  Files          13       13              
  Lines        1124     1115       -9     
==========================================
- Hits          494      492       -2     
+ Misses        630      623       -7     

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

@lauraporta lauraporta merged commit 8d2984a into main May 19, 2025
24 of 25 checks passed
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