Skip to content

feat(preprocessing): add binary closing and hole filling option - #12

Merged
404Simon merged 3 commits into
mainfrom
closing-and-hole-filling
Jun 11, 2026
Merged

feat(preprocessing): add binary closing and hole filling option#12
404Simon merged 3 commits into
mainfrom
closing-and-hole-filling

Conversation

@404Simon

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds optional morphological preprocessing steps to the vessel-analysis pipeline to improve skeletonization robustness, and exposes these new controls through the configuration model and the napari UI.

Changes:

  • Add optional binary closing and hole-filling steps (with a max hole-size threshold) before thinning, and surface the preprocessed binary in AnalysisResult.
  • Extend ExtractionConfig serialization/deserialization to include the new preprocessing settings.
  • Add napari widget controls to configure preprocessing and optionally display the preprocessed binary as an extra labels layer.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
vesskel/pipeline.py Implements optional morphological preprocessing and returns preprocessed_binary in the analysis result.
vesskel/config.py Adds new extraction settings (closing/fill holes/max hole size/show preprocessed) to the config model + (de)serialization.
vesskel/_napari.py Exposes new preprocessing options in the napari GUI and conditionally displays the preprocessed binary layer.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread vesskel/pipeline.py
Comment thread vesskel/pipeline.py
404Simon and others added 2 commits June 11, 2026 14:23
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@404Simon
404Simon merged commit 17656db into main Jun 11, 2026
2 checks passed
@404Simon
404Simon deleted the closing-and-hole-filling branch June 11, 2026 12:37
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