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

boxplot whisker fix #6749

Merged
merged 1 commit into from Oct 12, 2017

Conversation

Projects
None yet
2 participants
@d-rudolf
Contributor

d-rudolf commented Sep 25, 2017

Hello,
I'm using echarts 3.7.1 with Chromium 60.0.3112.113 and Ubuntu 16.04. LTS.
I slightly modified the Michelson-Morley data
[1, 2, 3, 4, 5, 6, 7, 8, 9, 850, 740, 900, 1070, 930, 850, 950, 980, 980, 880, 1000, 980, 930, 650, 760, 810, 1000, 1000, 960, 960] by adding the numbers 1 to 9 to the first dataset and got this plot:
michelson-morley experiment_old
Obviously, the whisker in this plot are wrong, because the dataset has no negative values for the speed of light. I rewrote the corresponding JavaScript function in the file echarts/extension/dataTool/prepareBoxplotData.js . With my code, the boxplot looks like this
michelson-morley experiment, which makes more sense.
I kindly request a git pull to fix the bug.

Best wishes,

Denis

@100pah 100pah merged commit 8475db1 into apache:master Oct 12, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment