[metrics-bugs] #26035 [Metrics/Statistics]: Streamline sample quantile types used in the various modules
Tor Bug Tracker & Wiki
blackhole at torproject.org
Mon Nov 5 21:23:47 UTC 2018
#26035: Streamline sample quantile types used in the various modules
--------------------------------+---------------------------
Reporter: karsten | Owner: karsten
Type: enhancement | Status: closed
Priority: Medium | Milestone:
Component: Metrics/Statistics | Version:
Severity: Normal | Resolution: fixed
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: irl | Sponsor: Sponsor13
--------------------------------+---------------------------
Changes (by karsten):
* status: merge_ready => closed
* resolution: => fixed
Comment:
Replying to [comment:26 karsten]:
> Replying to [comment:25 karsten]:
> > Replying to [comment:24 irl]:
> > > [...] 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.
>
> Still not closing until I get around to doing this.
Alright, #28342 now exists with many more ideas on this topic. Closing.
Thanks!
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/26035#comment:27>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the metrics-bugs
mailing list