Config graph_bound documentation inconsistent

torproject / nyx

Command-line monitor for Tor.

https://nyx.torproject.org/

GNU General Public License v3.0

123 stars 26 forks source link

Config graph_bound documentation inconsistent #22

Closed saizai closed 4 years ago

saizai commented 4 years ago

https://nyx.torproject.org/#configuration says graph_bound is default max_local and options are global_max, local_max, tight

So… which is it, max_local or local_max? And to double check: max_global or global_max?

atagar commented 4 years ago

Oops, thanks saizai. Good catch. Accidentally pushed a fix prior to realizing that this came with a pull request.

https://gitweb.torproject.org/nyx.git/commit/?id=c099fe3

saizai commented 4 years ago

Is gitweb.torproject the canonical repository, not github? If so, the github readme should say so prominently, since it's definitely not the default assumption.

atagar commented 4 years ago

Thanks Sai. Github is our canonical bug tracker, but the code mirrors our tor repository. This is generally moot for our contributors.

saizai commented 4 years ago

I suggest putting that in the README anyway, just for clarity.