Skip to content

Comments

ENH: IdentifySample - use algorithm class and add cancel and message feedback#1530

Merged
imikejackson merged 1 commit intoBlueQuartzSoftware:developfrom
imikejackson:topic/identify_sample_filter_updates
Feb 18, 2026
Merged

ENH: IdentifySample - use algorithm class and add cancel and message feedback#1530
imikejackson merged 1 commit intoBlueQuartzSoftware:developfrom
imikejackson:topic/identify_sample_filter_updates

Conversation

@imikejackson
Copy link
Contributor

@imikejackson imikejackson commented Feb 13, 2026

Algorithm Updates

  • Moved actual algorithm code to the Algorithm class.

Unit Testing

  • Added "Slice-by-slice execution" test with 3 SECTIONs (XY, XZ, YZ planes) using synthetic 5x5x3 geometry with a connected block and
    disconnected voxel
    • Added "UInt8 mask type" test using a 5x5x5 geometry with UInt8Array, testing both sample identification and hole filling

@imikejackson imikejackson force-pushed the topic/identify_sample_filter_updates branch 2 times, most recently from 63aa6a6 to 6bec6ca Compare February 16, 2026 21:37
@imikejackson imikejackson changed the title ENH: IdentifySample-move algorithm to Algorithm class and add cancel and message feedback ENH: IdentifySample - use algorithm class and add cancel and message feedback Feb 16, 2026
@imikejackson imikejackson enabled auto-merge (squash) February 16, 2026 21:39
@imikejackson imikejackson force-pushed the topic/identify_sample_filter_updates branch 3 times, most recently from 7306600 to 21abfea Compare February 18, 2026 01:45
@imikejackson imikejackson force-pushed the topic/identify_sample_filter_updates branch from 21abfea to ef0cc05 Compare February 18, 2026 16:28
@imikejackson imikejackson merged commit 27f2dba into BlueQuartzSoftware:develop Feb 18, 2026
6 checks passed
@imikejackson imikejackson deleted the topic/identify_sample_filter_updates branch February 18, 2026 19:30
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