Skip to content

Refactor image processing module#270

Merged
tieneupin merged 6 commits into
mainfrom
refactor-image-processing
Jun 23, 2026
Merged

Refactor image processing module#270
tieneupin merged 6 commits into
mainfrom
refactor-image-processing

Conversation

@tieneupin

@tieneupin tieneupin commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

The image processing module has grown exceptionally large as more image registration algorithms are developed and added. The complexity of the image registration algorithms necessitates multiple helper functions, and when they are all housed within the same module, function name real estate becomes difficult to manage, and has led to function design choices that make reviewing and understanding them difficult (e.g. having multiple functions nested within the main image alignment function).

This PR refactors the image processing module into smaller sub-modules, all stored under the cryoemservices.util.image_processing folder. The image alignment/registration functions are each moved into their own modules, and the generic, shared functions are kept as-is, with the module being renamed as shared. The tests have been migrated to reflect this new layout as well.

This PR is purely code migration, with no changes having been made to the function logic. These will be handled in subsequent PRs, as part of the effort to keep the changes scoped and understandable.

@codecov

codecov Bot commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.10675% with 50 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.12%. Comparing base (944a17a) to head (a0712ec).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #270      +/-   ##
==========================================
+ Coverage   83.08%   83.12%   +0.04%     
==========================================
  Files          72       75       +3     
  Lines        9265     9288      +23     
  Branches     1264     1264              
==========================================
+ Hits         7698     7721      +23     
  Misses       1075     1075              
  Partials      492      492              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tieneupin tieneupin self-assigned this Jun 12, 2026
@tieneupin tieneupin added the enhancement New feature or request label Jun 12, 2026
@tieneupin tieneupin marked this pull request as ready for review June 12, 2026 16:23
@tieneupin tieneupin merged commit 8e3d94d into main Jun 23, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants