Skip to content
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.

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

Add custom clip Filter classes #1376

Merged
merged 1 commit into from
May 31, 2019
Merged

Add custom clip Filter classes #1376

merged 1 commit into from
May 31, 2019

Conversation

njmei
Copy link
Collaborator

@njmei njmei commented May 24, 2019

This commit adds two custom clip Filter classes called:
ClipValueToZero and ClipPercentileToZero

They operate by clipping to a minimum value (or min percentile value)
and then subtracting the entire image with that minimum value.

This will allow us to create/run recipes for our pipeline(s).

@codecov-io
Copy link

codecov-io commented May 24, 2019

Codecov Report

Merging #1376 into master will increase coverage by 0.08%.
The diff coverage is 96.92%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1376      +/-   ##
==========================================
+ Coverage    89.4%   89.49%   +0.08%     
==========================================
  Files         149      151       +2     
  Lines        5416     5481      +65     
==========================================
+ Hits         4842     4905      +63     
- Misses        574      576       +2
Impacted Files Coverage Δ
starfish/core/image/_filter/clip_value_to_zero.py 96.55% <96.55%> (ø)
...fish/core/image/_filter/clip_percentile_to_zero.py 97.22% <97.22%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d3f3a84...ba40d86. Read the comment docs.

@neuromusic neuromusic requested review from ttung and ambrosejcarr May 30, 2019 20:53
Copy link
Collaborator

@ttung ttung left a comment

Choose a reason for hiding this comment

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

Mostly fine, please note the comments.

starfish/core/image/_filter/clip_value_to_zero.py Outdated Show resolved Hide resolved
starfish/core/image/_filter/clip_value_to_zero.py Outdated Show resolved Hide resolved
starfish/core/image/_filter/clip_value_to_zero.py Outdated Show resolved Hide resolved
This is a combination of 2 commits:

1) This commit adds two custom clip `Filter` classes called:
   ClipValueToZero and ClipPercentileToZero

   They operate by clipping to a minimum value (or min percentile value)
   and then subtracting the entire image with that minimum value.

2) Fix whitespace and type annotations
@njmei
Copy link
Collaborator Author

njmei commented May 31, 2019

Okay, should be ready for merge now.

@ttung ttung merged commit 8383e80 into spacetx:master May 31, 2019
@njmei njmei deleted the njmei-add-custom-clip-filters branch May 31, 2019 15:42
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.

3 participants