[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #26857 [Metrics/Website]: Add new page with specifications for reproducing graphs and tables



#26857: Add new page with specifications for reproducing graphs and tables
-----------------------------+-----------------------------------
 Reporter:  karsten          |          Owner:  karsten
     Type:  enhancement      |         Status:  needs_information
 Priority:  Medium           |      Milestone:
Component:  Metrics/Website  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:                   |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:  irl              |        Sponsor:  Sponsor13
-----------------------------+-----------------------------------
Changes (by karsten):

 * status:  needs_revision => needs_information


Comment:

 Replying to [comment:8 irl]:
 > The anchors/headings, I mean:
 >
 > {{{
 > <h4>Step 3: Estimate fraction of reported directory-request statistics
 > <a href="#relay-users-fraction" name="relay-users-fraction"
 class="anchor">#</a></h4>
 > }}}
 >
 > While other `<h4>` headings don't have the anchors.

 Ah, the reason was that we were only referring to those two `<h4>`
 headings, not to any others. But I see the point of trying to be
 consistent. I removed those anchors and adapted the links in
 [https://gitweb.torproject.org/karsten/metrics-
 web.git/commit/?h=task-26857&id=62e60f0aa21e93ed51ffe5abe7f835383c01478e
 commit 62e60f0].

 > > to avoid including statistics in the aggregation that have very likely
 been reported and processed before
 >
 > Does it make sense to say instead "to avoid including statistics in the
 aggregation that are likely to not be relevant to the time period we are
 calculating for"?

 While that would work, too, I tried a slightly different change (new text
 in italics): "that have very likely been reported ''in earlier
 descriptors'' and processed before." See
 [https://gitweb.torproject.org/karsten/metrics-
 web.git/commit/?h=task-26857&id=22c304dce7c60d72ab10d1c7b44f79f77a7a7059
 commit 22c304d].

 > Other than that it is all looking OK.

 Okay, great, I went ahead and rebased/squashed these commits into master
 and deployed them. This page is certainly not "done", but I think it's
 done enough now to be shared publicly.

 Another part that remains to be done is to fully integrate this page into
 the rest of Tor Metrics by adding links. Let's first ask more folks to
 review what's there, though. Setting to needs_information until then.

 Thanks!

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/26857#comment:9>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs