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

Improve calculation of y-axis maximum value #132

Closed
airblade opened this issue Jun 3, 2013 · 6 comments
Closed

Improve calculation of y-axis maximum value #132

airblade opened this issue Jun 3, 2013 · 6 comments
Assignees
Milestone

Comments

@airblade
Copy link

airblade commented Jun 3, 2013

I felt the y-axis maximum value was often too large.

Now a maximum data value of 1431 gives a maximum y-axis value of 1500
instead of 2000.

airblade@3cdecb1

@fulldecent
Copy link
Contributor

I am not able to reproduce this problem using the new version release candidate.

Could you please check if this has been fixed and close or create a JSBin showing the issue.

Thank you

@airblade
Copy link
Author

airblade commented Nov 7, 2014

Sorry for the slow reply.

I tested this again just now against master and the "problem" is still there. I created a JSBin with extracts from the Chart.js source:

http://jsbin.com/ramoneyubu/1/edit?js,output

@ernestpob
Copy link

Hi,
I used the bar chart and after one of my values reached a 1000, the bar messes up values and transitively the chart itself.
the code is below:

    var barChartData = {
        labels : ["January","February","March","April","May","June","July","August"],
        datasets : [
            {
                fillColor : "rgba(220,220,220,0.5)",
                strokeColor : "rgba(220,220,220,1)",
                                        data : ["1002","7","147","66","1","448","136","190"]
            }
        ]           
    }

var myLine = new Chart(document.getElementById("canvas").getContext("2d")).Bar(barChartData);

@etimberg etimberg added this to the 1.1 milestone Jan 8, 2015
@etimberg
Copy link
Member

etimberg commented Jan 8, 2015

I think this is a good candidate for fixing in 1.1

@fulldecent
Copy link
Contributor

Moved to milestone Axis. Not saying this won't be completed for 1.1, just trying to keep those issues closely associated

@fulldecent fulldecent modified the milestones: Rewrite axis/scale code, 1.1 Apr 16, 2015
@etimberg etimberg modified the milestones: v2.0, Rewrite axis/scale code May 20, 2015
@etimberg etimberg self-assigned this May 20, 2015
@derekperkins
Copy link
Contributor

@airblade Huzzah! The first alpha of Chart.js 2.0 has landed and should fix this issue. Check out the release and try it out! We've got a lot of momentum right now, so please help us test so we can launch 2.0 Gold by the end of the month.
https://github.com/nnnick/Chart.js/releases/tag/v2.0-alpha

I'm closing this issue for now, but if you have implementation questions or find bugs, please create a jsfiddle and post the link here and we'll reopen this issue and get it fixed.

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

No branches or pull requests

5 participants