Opened 4 months ago

Closed 4 months ago

#34217 closed enhancement (wontfix)

Include partial downloads in existing TTLB graphs

Reported by: karsten Owned by: metrics-team
Priority: Medium Milestone:
Component: Metrics/Onionperf Version:
Severity: Normal Keywords:
Cc: metrics-team, acute, phw Actual Points:
Parent ID: #33327 Points:
Reviewer: Sponsor: Sponsor59

Description

With #26673 being resolved we should include partial downloads in existing TTLB graphs. Maybe there's a way to put less emphasis on the different file sizes and combine the various partial downloads in a single TTLB graph.

Child Tickets

Change History (2)

comment:1 Changed 4 months ago by karsten

Parent ID: #33327

comment:2 Changed 4 months ago by karsten

Resolution: wontfix
Status: newclosed

In retrospect, let's not do this. The main reason why we included partial downloads in Tor Metrics graphs is that we wanted to continue the 50 KiB and 1 MiB graphs even after stopping those downloads. And we're going to keep doing that.

But there's no good reason to plot exactly these partial download times in OnionPerf, where we typically plot experimental measurements or comparisons between long-running and experimental measurements. We'll have TTFB and TTLB.

Closing as wontfix, even though we can always come back to this and do it or a variant of it.

Note: See TracTickets for help on using tickets.