[tor-bugs] #2603 [Metrics]: Remove dead parts from Torperf directory

Tor Bug Tracker & Wiki torproject-admin at torproject.org
Fri Feb 25 07:18:38 UTC 2011


#2603: Remove dead parts from Torperf directory
--------------------------------------+-------------------------------------
 Reporter:  karsten                   |          Owner:  karsten     
     Type:  defect                    |         Status:  needs_review
 Priority:  normal                    |      Milestone:              
Component:  Metrics                   |        Version:              
 Keywords:  TorPerfIteration20110305  |         Parent:              
   Points:                            |   Actualpoints:              
--------------------------------------+-------------------------------------

Comment(by Sebastian):

 Replying to [comment:4 karsten]:
 > Replying to [comment:3 Sebastian]:
 > > Looks good mostly. In the README, maybe the link between the different
 files could be made clearer?
 >
 > I think the real problem here is that we have too many independent
 scripts that may or may not be optional, depending on what you're doing.
 I think we should implement #2565 and get rid of most of the files
 described in the `README`.  This cleanup is mostly for people looking into
 Torperf before that happens, but it's probably not perfect.

 Ah. I made everything into its own script so it would be easy to verify
 each on its own, and so that you only need to run those parts that you
 actually need. But if a single script does all that'd work too of course.

 >
 > > extra_stats.py really is optional, too; and a note could be added that
 if it is used, consolidate_stats.py will help make sense of the output.
 >
 > `extra_stats.py` may be optional, but we included it in the basic setup
 instructions in `measurements_HOWTO`, so saying it's optional might
 confuse people.  Also, `consolidate_stats.py` is broken right now, because
 we changed the `.extradata` file format (I just opened #2626 for that).

 We should not break it next time before it is obsolete ;)

 > > On the other hand, maybe it's not that big a deal because people
 running torperf will already know what kind of experiment they want to do.
 hm.
 >
 > Right, whoever uses Torperf these days will have to know what they want
 to do.  We should change that (#2565).

 Hrm. Sounds like people should think about what they want to do before
 wasting bandwidth.

 >
 > Sounds like we can merge?

 Sure.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/2603#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list