Skip to content

FILT: Extract Feature Boundaries 2D filter implemented #1510

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
imikejackson wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

FILT: Extract Feature Boundaries 2D filter implemented #1510

imikejackson wants to merge 1 commit into from

Conversation

@imikejackson
Copy link
Contributor

@imikejackson imikejackson commented Jan 21, 2026 •
edited
Loading

  • Documentation Updated

  • Unit test implemented

  • New unit test data archive file uploaded to GitHub and Sync'ed

  • 1 Unit test to test output from the filter against known exemplar set of data

  • 1 Unit test to test invalid input code paths that are specific to a filter. Don't test that a DataPath does not exist since that test is already performed as part of the SelectDataArrayAction.

Code Cleanup

  • No commented-out code (rare exceptions to this is allowed)
  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)
  • Updated API documentation (or API not changed)
  • Added license to new files (if any)
  • Added example pipelines that use the filter
  • Classes and methods are properly documented

@imikejackson imikejackson force-pushed the topic/extract_feature_boundaries_2d branch 3 times, most recently from d757a3d to 281a7d8 Compare January 21, 2026 19:58
@imikejackson
FILT: Extract Feature Boundaries 2D filter implemented
e9e1398 
- Documentation Updated
- Unit test implemented
- New unit test data archive file uploaded to GitHub and Sync'ed
@imikejackson imikejackson force-pushed the topic/extract_feature_boundaries_2d branch from 281a7d8 to e9e1398 Compare January 23, 2026 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JDuffeyBQ JDuffeyBQ Awaiting requested review from JDuffeyBQ

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@imikejackson