/speed

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

add Histogram #30

Merged
merged 11 commits into from Aug 24, 2016
+996 −0

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@suyash @coveralls
@suyash
Collaborator

suyash commented Aug 12, 2016 • edited

the implementation is backed by https://github.com/codahale/hdrhistogram
@suyash
Collaborator

suyash commented Aug 12, 2016

speed:
speed

pmval:
pmval
@coveralls

coveralls commented Aug 12, 2016 • edited

Coverage Status

Coverage decreased (-7.003%) to 76.978% when pulling 39b9fa6 on histogram into 50bde18 on master.

suyash added some commits Aug 12, 2016

@suyash
vendor: add codahale hdrhistogram
c682440
@suyash
instance: add a Keys method to Instances
fa3d868
@suyash
metrics, client: add a basic histogram 
backed by coda hale's hdrhistogram

this adds a basic histogram using a pcp Instance Metric and a Histogram
from hdrhistogram. The metric has InstantSemantics, DoubleType and OneUnit.
The supported operations are

- record single value
- record multiple occurrences of same value
- get max value recorded so far
- get min value recorded so far
- get mean
- get variance
- get standard deviation
814a5bf
@suyash
examples: add a basic histogram example
b3e273b
@suyash
metrics: fix lint errors
ecd8033
@suyash
metrics: histogram: refactor update and add Percentile

All checks have failed

3 failing checks
continuous-integration/travis-ci/pr — The Travis CI build failed
Details
continuous-integration/travis-ci/push — The Travis CI build failed
Details
coverage/coveralls — Coverage decreased (-6.6%) to 77.65%
Details
6483952
@coveralls

coveralls commented Aug 23, 2016 • edited

Coverage Status

Coverage decreased (-6.6%) to 77.65% when pulling 6483952 on histogram into d71c978 on master.

suyash added some commits Aug 23, 2016

@suyash
metrics: add significant figures to Histogram constructor
bd840da
@suyash
example: basic_histogram: pass significant figures during construction

Some checks were not successful

1 failing and 2 successful checks
coverage/coveralls — Coverage decreased (-7.3%) to 76.874%
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
continuous-integration/travis-ci/push — The Travis CI build passed
Details
bc50ef7
@coveralls

coveralls commented Aug 23, 2016 • edited

Coverage Status

Coverage decreased (-7.3%) to 76.874% when pulling bc50ef7 on histogram into d71c978 on master.

suyash added some commits Aug 24, 2016

@suyash
metrics: histogram: add Buckets
f21f64e
@suyash
client: add histogram tests

All checks have failed

3 failing checks
continuous-integration/travis-ci/pr — The Travis CI build failed
Details
continuous-integration/travis-ci/push — The Travis CI build failed
Details
coverage/coveralls — Coverage decreased (-2.7%) to 81.463%
Details
6635d0e
@coveralls

coveralls commented Aug 24, 2016

Coverage Status

Coverage decreased (-2.7%) to 81.463% when pulling 4b89053 on histogram into d71c978 on master.
@suyash
client: fix lint errors in tests

All checks have passed

3 successful checks
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
continuous-integration/travis-ci/push — The Travis CI build passed
Details
coverage/coveralls — Coverage increased (+0.1%) to 81.605%
Details
80591f0
@coveralls

coveralls commented Aug 24, 2016 • edited

Coverage Status

Coverage decreased (-2.6%) to 81.605% when pulling 80591f0 on histogram into d71c978 on master.

@suyash suyash merged commit 8b3fa33 into master Aug 24, 2016
3 checks passed

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.1%) to 81.605%
Details

@suyash suyash deleted the histogram branch Aug 24, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment