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

Re: [tor-bugs] #33258 [Metrics]: Add CSV file export of graphed data



#33258: Add CSV file export of graphed data
-----------------------------------------+------------------------------
 Reporter:  karsten                      |          Owner:  metrics-team
     Type:  enhancement                  |         Status:  needs_review
 Priority:  Medium                       |      Milestone:
Component:  Metrics                      |        Version:
 Severity:  Normal                       |     Resolution:
 Keywords:  metrics-team-roadmap-2020Q1  |  Actual Points:
Parent ID:  #33327                       |         Points:  1
 Reviewer:  anarcat                      |        Sponsor:  Sponsor59
-----------------------------------------+------------------------------

Comment (by robgjansen):

 Karsten, I think all of the changes you have made here are improvements
 over the original graphs. I copied the original code from scripts I was
 using for Shadow experiments, which is why some of the plots probably
 don't make sense anymore outside of Shadow.

 > we should expect developers wanting to play with the underlying data and
 feeding it into their own tools

 This was exactly my sentiment too when I added the OnionPerf plotting; I
 thought it would be more useful to start with the code I had been using
 for Shadow than starting with nothing. :)

 Also, I ended up copying a lot of the TGen parsing/plotting stuff into a
 python package in [https://github.com/shadow/tgen/tree/master/tools the
 tgen repository]. I thought it made more sense for the code that plots
 tgen logs to coexist and be synchronized with tgen itself. This means we
 now have two different tools to parse/plot tgen results, which I think is
 fine, but let me know if you don't want "us" to be maintaining two
 separate code-bases for this. I don't mind if you completely change the
 way OnionPerf is plotting things so that the plots are most useful for
 OnionPerf, and we'll make the tgen plots useful for Shadow. Does that
 approach make sense?

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