[metrics-bugs] #26035 [Metrics/Statistics]: Streamline sample quantile types used in the various modules

Tor Bug Tracker & Wiki blackhole at torproject.org
Thu Oct 25 09:38:50 UTC 2018


#26035: Streamline sample quantile types used in the various modules
--------------------------------+-----------------------------
 Reporter:  karsten             |          Owner:  karsten
     Type:  enhancement         |         Status:  merge_ready
 Priority:  Medium              |      Milestone:
Component:  Metrics/Statistics  |        Version:
 Severity:  Normal              |     Resolution:
 Keywords:                      |  Actual Points:
Parent ID:                      |         Points:
 Reviewer:  irl                 |        Sponsor:  Sponsor13
--------------------------------+-----------------------------

Comment (by karsten):

 Replying to [comment:24 irl]:
 > I think this is ready to go, [...]

 Thanks for the review! Pushed to master, deploying now, step by step.
 Leaving this ticket open until everything's deployed.

 > [...] but, we should create a new ticket for generalising the
 computePercentiles function with generics.

 I'll create a slightly different ticket before closing this one: we should
 share more code between modules, and a generic `computePercentiles()`
 would be one piece of code to consider here. I wouldn't want to merge a
 patch that starts with this part of the code. I'd want us to approach this
 from a top-down perspective where we generalize similar functionality and
 use it in all modules to reduce the overall amount of code and likelihood
 of bugs.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/26035#comment:25>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the metrics-bugs mailing list