The patch also adds a table of contents to onionperf's README, which makes it easier to browse.
I wonder if we should merge README.md with onionperf.rst. The two files contain redundant installation instructions and I found it confusing that more documentation was hidden in the source code directory.
The patch also adds a table of contents to onionperf's README, which makes it easier to browse.
The patch looks good, I think this can be merged!
I wonder if we should merge README.md with onionperf.rst. The two files contain redundant installation instructions and I found it confusing that more documentation was hidden in the source code directory.
These two were intended to serve different purposes. The readme was meant to be a quick reference, and the docs an in-depth guide which includes module documentation (sphinx autodoc can be used to build documentation from docstrings) and more exotic deployment options (for say, researchers). The docs are currently served here: https://onionperf.torproject.org
Oh, I hadn't seen that this is ready to be merged. I guess we need to copy metrics-team in order to get updates sent to metrics-bugs@.
Rebased and pushed to master!
I agree that having documentation in several places is a bit confusing. That is on my list, but I'll create a ticket to discuss any changes before making them.
Moving acute's 0.2 actual points to the Actual Points field (yes, that is confusing). phw, please add any actual points of yours to those.
Closing. Thanks!
Trac: Resolution: N/Ato fixed Status: merge_ready to closed Actualpoints: N/Ato 0.1 Cc: acute to acute, metrics-team Points: 0.3 to 0.2