-
Notifications
You must be signed in to change notification settings - Fork 579
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
Bug fix for smallest_dot
in sc.pl.dotplot()
#1771
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1771 +/- ##
=======================================
Coverage 71.21% 71.22%
=======================================
Files 92 92
Lines 11181 11181
=======================================
+ Hits 7963 7964 +1
+ Misses 3218 3217 -1
|
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.
Thanks for the PR! Yeah, this looks like a typo, and your fix looks good.
Could you update the reference plot for the tests? This will just entail replacing the reference figure with the new figure generated by the failing test? This may need to happen for all the tests which use smallest_dot
.
Hi @ivirshup , I replaced the one test image that was causing a failure, as you suggested. (And I checked to make sure the image makes sense... it does...) I think this should do it. |
Looks good, thanks! |
Co-authored-by: Stephen Fleming <sjfleming@users.noreply.github.com>
Solves #1770