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

Fix zero pixels #956

Merged
merged 1 commit into from
Mar 23, 2023
Merged

Fix zero pixels #956

merged 1 commit into from
Mar 23, 2023

Conversation

cliu72
Copy link
Contributor

@cliu72 cliu72 commented Mar 23, 2023

If you haven't already, please read through our contributing guidelines before opening your PR

What is the purpose of this PR?

After removing the pixel thresholding step (to filter out low pixels), we never added back in the filter for pixels that were zero for all markers. Added that back in.

How did you implement your changes

Added one line to remove pixels that are 0 for all markers before row normalization.

Remaining issues

None

@cliu72 cliu72 requested a review from ngreenwald March 23, 2023 00:33
@ngreenwald ngreenwald merged commit f6e0d2a into main Mar 23, 2023
@ngreenwald ngreenwald deleted the fix_zero_pixels branch March 23, 2023 02:17
@srivarra srivarra added the bug Something isn't working label Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants