Skip to content
This repository has been archived by the owner on Nov 17, 2020. It is now read-only.

Fix for Rounding issue with moving averages . #60

Merged
merged 1 commit into from Sep 4, 2015
Merged

Fix for Rounding issue with moving averages . #60

merged 1 commit into from Sep 4, 2015

Conversation

Gsantomaggio
Copy link
Member

Fixes #35.

@michaelklishin
Copy link
Member

Please specify what issue this is in the description.

@michaelklishin michaelklishin self-assigned this Sep 3, 2015
@michaelklishin michaelklishin added this to the n/a milestone Sep 3, 2015
@Gsantomaggio
Copy link
Member Author

Hi @michaelklishin ,

Fixed this function  fmt_num_thousands since by default it doesn’t work correctly with numbers with point, like this ( 1234.5678 )

Renamed the original fmt_num_thousands to fmt_num_thousands_unfixed  

The new version  uses toFixed(0)  to avoid this behavior

See the issue #35

@michaelklishin michaelklishin merged commit 80c9e47 into stable Sep 4, 2015
@dumbbell dumbbell mentioned this pull request Sep 8, 2015
Closed
@dumbbell dumbbell deleted the rabbitmq-management-35 branch January 3, 2018 08:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@michaelklishin michaelklishin

Labels
None yet
Projects
None yet
Milestone
n/a
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Gsantomaggio @michaelklishin