Skip to content

Fix label padding for large numbers - #4

Open
brdu wants to merge 1 commit into
kroitor:masterfrom
brdu:bd_fix_labels
Open

Fix label padding for large numbers #4
brdu wants to merge 1 commit into
kroitor:masterfrom
brdu:bd_fix_labels

Conversation

@brdu

@brdu brdu commented Dec 4, 2017

Copy link
Copy Markdown

Add this code to test.js to reproduce

var s3 = []
for (var i = 0; i < width; i++)
    s3[i] = (2000 * Math.cos (i * ((Math.PI * 8) / width))) + 10000 // values range from 9000 to 11000
console.log (asciichart.plot (s3, { height: 10, width: 60 }))

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-3.2%) to 96.774% when pulling c1aef00 on brdu:bd_fix_labels into be5de29 on kroitor:master.

@kroitor

kroitor commented Dec 5, 2017

Copy link
Copy Markdown
Owner

Thank you very much for this commit! I'll merge it asap!

@kroitor kroitor self-assigned this Dec 5, 2017
@kroitor kroitor mentioned this pull request Dec 10, 2017
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

Successfully merging this pull request may close these issues.

3 participants