Skip to content

Conversation

@lochhh
Copy link
Member

@lochhh lochhh commented Dec 11, 2025

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other

Why is this PR needed?
This PR closes #723

What does this PR do?
This PR

References

#723

How has this PR been tested?

Tests have been updated for refactors; tests continue to pass.

Is this a breaking change?

No

Does this PR require an update to the documentation?

docstrings have been updated accordingly.

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

codecov bot commented Dec 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (e41a5d0) to head (141a026).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #721   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           34        34           
  Lines         2109      2104    -5     
=========================================
- Hits          2109      2104    -5     

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

Copy link
Member

@niksirbi niksirbi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for undertaking this @lochhh.

It must have been a massive amount of work, especially the bits about ROIs and broadcasting.

As far as I can tell, everything works fine now. I'm approving this but will leave you to deal with the merging. This PR is probably going to lead to many merge conflicts, especially with PR 727, but it's probably best to bite the bullet and merge this one soon.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 8, 2026

@lochhh lochhh added this pull request to the merge queue Jan 8, 2026
Merged via the queue into main with commit 96c28a7 Jan 8, 2026
29 of 31 checks passed
@lochhh lochhh deleted the mypy-errors branch January 8, 2026 17:04
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.

Uncaught mypy errors

3 participants