#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
_______________________________________________ tor-bugs mailing list tor-bugs@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs