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
Implement TissueMask
mask for Tile
with type dispatcher
#313
Conversation
Codecov Report
@@ Coverage Diff @@
## master #313 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 17 17
Lines 1272 1284 +12
Branches 124 124
=========================================
+ Hits 1272 1284 +12
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please address the minor comment
This pull request fixes 1 alert when merging 959b712 into f86c65e - view on LGTM.com fixed alerts:
|
hey @ernestoarbitrio what do I do with this CodeFactor issue?
I wanted that method for the sake of the documentation, can I achieve the same result in another way? |
This pull request fixes 1 alert when merging 7df6bac into f86c65e - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging f187c3c into f86c65e - view on LGTM.com fixed alerts:
|
8da21db
to
a1cc719
Compare
This pull request fixes 1 alert when merging a1cc719 into f86c65e - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging 6c3ae1a into f86c65e - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging 138c3f2 into f86c65e - view on LGTM.com fixed alerts:
|
138c3f2
to
c557145
Compare
This pull request fixes 1 alert when merging c557145 into a7dc352 - view on LGTM.com fixed alerts:
|
Description
Now is possible to use the
TissueMask
mask also onTile
s:Types of Changes
Issues Fixed or Closed by This PR
TissueMask
mask onTile
#311Checklist