improve plotting to make better use of y range #42
Conversation
|
Just wondering if you have questions or comments on this one. |
|
Hi otteresk, sorry for the delay I have been very busy recently. I will look through your changes as soon as possible. |
|
Hi Orbiter, no problem. If you would like to see the new version you can check on my node: Kind regards, Andreas |
Get my fork synced #4
|
Hi Orbiter et al. |
|
Hi, without patch |
| @@ -168,7 +168,7 @@ private void drawVerticalScale(final boolean left, final int scale, final int pi | |||
| } | |||
| setColor(colorNaming); | |||
| line(x - 3, y, x + 3, y, 100); | |||
| s1 = (s >= 1000000 && s % 10000 == 0) ? Integer.toString(s / 1000000) + "M" : (s >= 1000 && s % 1000 == 0) ? Integer.toString(s / 1000) + "K" : Integer.toString(s); | |||
| s1 = (s >= 1000000 && s % 1000000 == 0) ? Integer.toString(s / 1000000) + "M" : (s >= 1000 && s % 1000 == 0) ? Integer.toString(s / 1000) + "K" : Integer.toString(s); | |||
There was a problem hiding this comment.
fyi: this bugfix will be applied for sure
|
Thanks for the test and feedback, Burkhard. |
#42 thanks to otteresk (https://github.com/otteresk)
|
Should be fixed now. |
Get my fork synced #5
|
I really have to get used to git(hub) :-) |
|
Hi Burkhard, any chance you find time for another test? |
New calculation of y range to better handle big values with small variations (e.g. no. documents)