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] Box plot: Handle situation when quantiles can't be computed #1742

Merged
merged 1 commit into from
Nov 11, 2016

Conversation

janezd
Copy link
Contributor

@janezd janezd commented Nov 11, 2016

Issue

If quantiles can't be computed, the widget sets them to -1 and shows a mess (issue #1648).

Description of changes

Set them to None and skip drawing.

Includes
  • Code changes
  • Tests
  • Documentation

@codecov-io
Copy link

codecov-io commented Nov 11, 2016

Current coverage is 88.87% (diff: 100%)

Merging #1742 into master will not change coverage

@@             master      #1742   diff @@
==========================================
  Files            82         82          
  Lines          8819       8819          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           7838       7838          
  Misses          981        981          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update ad4c887...ea9fd27

@janezd janezd force-pushed the mosaic-fix-no-quantiles branch from 59a5621 to ea9fd27 Compare November 11, 2016 09:45
@janezd janezd changed the title Mosaic: Handle situation when quantiles can't be computed [FIX] Mosaic: Handle situation when quantiles can't be computed Nov 11, 2016
@janezd janezd changed the title [FIX] Mosaic: Handle situation when quantiles can't be computed [FIX] Box plot: Handle situation when quantiles can't be computed Nov 11, 2016
@lanzagar lanzagar merged commit 1c705a6 into biolab:master Nov 11, 2016
@janezd janezd deleted the mosaic-fix-no-quantiles branch April 5, 2019 17:31
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