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

Missing a piece of data to the left #785

Closed
oxodpavel opened this issue Mar 2, 2016 · 4 comments
Closed

Missing a piece of data to the left #785

oxodpavel opened this issue Mar 2, 2016 · 4 comments

Comments

@oxodpavel
Copy link

Hi!

When scrolling the graph horizontally, it sometimes misses a piece of line to the left of the most left point of the view box. That is, a piece of line between Y axis and the first point on the graph

I do not if this can be assumed as an issue or a default behavior or I can configure it from code, but sometimes there may be a misunderstanding on whether there is some data to the left of not.

test_app

@liuxuan30
Copy link
Member

your screenshot sesm not reflect the problem, could you mark where is missing? or a comparision

@oxodpavel
Copy link
Author

Here is another screenshot with marked area. To the left of the point there is some preceding data, but the graph line piece is missed. And when you move horizontal axis it sometimes becomes visible or invisible
app

@danielgindi
Copy link
Collaborator

@PhilJay can you take a look at the PR?

@PhilJay
Copy link
Collaborator

PhilJay commented Mar 17, 2016

If the PR introduces correct behaviour or not depends on the implementation of the round(...) method.
What we basically want is to display the lowest VISIBLE index.

This means if the edge of the chart is at 0.01 or 1, the lowest visible x-index returned by the method should be 1.

According to this post, the round method does is round up for everything >= 0.5
This would introduce incorrect behaviour for values between 0.01 and 0.49, because those values would be rounded down but actually should be rounded up.

I think the ceil(...) method should be used to force rounding up.

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

No branches or pull requests

4 participants