zeni/go-charts
1
0
Fork 0
Code Issues Pull requests Projects Releases 47 Packages Wiki Activity Actions

Change ticks to avoid values impacting each other

Browse source 
The recently introduced logic has an incorrect understanding of the `unit` parameter.
This would result in too many ticks being outputted, particularly as datasets got larger.
This fixes it by re-calculating the tick count using the `unit` param as originally intended.
This commit is contained in:
Mike Jensen 2023-05-05 09:44:09 -06:00
parent 19173dfd37
commit c810369730
No known key found for this signature in database
GPG key ID: 2C909C9AAFB1047D
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
painter.go
View file

@ -615,7 +615,8 @@ func (p *Painter) TextFit(body string, x, y, width int, textAligns ...string) ch
return output return output
} }
func isTick(totalRange int, numTicks int, index int) bool { func isTick(totalRange int, unit int, index int) bool {
numTicks := (totalRange / unit) + 1
step := float64(totalRange-1) / float64(numTicks-1) step := float64(totalRange-1) / float64(numTicks-1)
for i := int(float64(index) / step); i < numTicks; i++ { for i := int(float64(index) / step); i < numTicks; i++ {
value := int((float64(i) * step) + 0.5) value := int((float64(i) * step) + 0.5)
@ -737,7 +738,7 @@ func (p *Painter) MultiText(opt MultiTextOption) *Painter {
} }
} else { } else {
if index == len(opt.TextList) - 1 { if index == len(opt.TextList) - 1 {
x = start - box.Width() x = start - box.Width() + 10
} else { } else {
x = start - box.Width()>>1 x = start - box.Width()>>1
} }