Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/685#issuecomment-179662685
I have slightly different view, what I think original idea behind using
`var integerFormatter = d3.format(',.1d');`
was to display number like `123123123` to `123,123,123`, but by
`d3.round(d, 3)`
it looses its formatting, and in my suggestion if you use
`var integerFormatter = function(x){return d3.format(',')(d3.round(x,
3));};`
it retains formatting and have decimal precision as well, kind of hacky but
it works.
Also its integerFormatter is referenced in paragraph.controller.js.html as
well, do you think this should be changed as well ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
