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

Re: [tor-bugs] #23169 [Metrics/Website]: Explain why metrics are important and what we do to make sure they're safe



#23169: Explain why metrics are important and what we do to make sure they're safe
-----------------------------+------------------------------
 Reporter:  arma             |          Owner:  irl
     Type:  defect           |         Status:  needs_review
 Priority:  Medium           |      Milestone:
Component:  Metrics/Website  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:  metrics-2018     |  Actual Points:
Parent ID:                   |         Points:  2
 Reviewer:  iwakeh, karsten  |        Sponsor:
-----------------------------+------------------------------

Comment (by irl):

 Replying to [comment:24 iwakeh]:
 > The dot file should not be in the 'web/images' folder as it is only the
 source for the png.  The png should always be committed (same process as
 with our generated spec jsps, for example), because these depend on the
 dot implementation used.  Even with a future Ant task at hand that limits
 dependencies for building the war file.  Regarding dot file location: what
 about `src/main/resources/doc/ecosystem.dot` or
 `src/main/resources/doc/ecosystem/ecosystem.dot`?

 Having reviewed the man page for dot, I noticed the standard file
 extension is actually .gv so I've changed that and moved the source to
 `src/main/resources/graphviz`. The idea would be that this serves as a
 common directory for any graphviz source files and then they are installed
 into `src/main/resources/web/images` with the "generate-graphviz-pngs" Ant
 task (added in a separate commit).

 > A second thought about this graph, which I like and understand, but it
 might cause confusion for the 'uninitiated' as it is not really described
 fully.  And, the metrics-lib oval is kind of dangling in the air and the
 "via metrics-lib" label used three times.  I first thought of making it a
 different shape (see attachment),
 > [[Image(https://trac.torproject.org/projects/tor/attachment/ticket/23169
 /another-overview.png)]]
 > but then noticed that this also is not at all self-explanatory.  So,
 maybe leave the picture out unless we describe the graph a little more?

 I've removed the floating metrics-lib and instead added 3 smaller boxes on
 the paths between CollecTor and the consuming services. I've also added an
 additional paragraph above the image that should help to explain it
 further. If people want to learn additional information, they should read
 the relevant pages that are linked in the text, this is only meant to be a
 quick primer.

 > Should we think about removing Schneier's quote from the header now (as
 was requested before)?
 > It's on the about page now.

 I've created #25411 (as a subticket of #25404) for this.

 Changes are rebased and pushed to my
 [[https://gitweb.torproject.org/user/irl/metrics-
 web.git/log/?h=task/23169|task/23169]] branch.

 I would be particularly interested in feedback on my Ant, as I haven't
 done much Ant and learning what I'm doing wrong is good.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/23169#comment:26>
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