Opened 8 years ago

Closed 8 years ago

#6692 closed enhancement (implemented)

Displaying more information when using 'group by' feature

Reported by: cypherpunks Owned by:
Priority: Medium Milestone:
Component: Metrics/Compass Version:
Severity: Keywords:
Cc: gsathya Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


When using the 'group by' feature (e.g. [1]) the columns 'Exit', 'Guard' and 'AS name' are basically empty.

I find it useful that you display the number of relays in the nickname column and think that could be extended to these other columns to answer the following questions:

  • How many relays with the exit flag are in a given group?
  • How many relays with the guard flag are in a given group?
  • How many distinct AS numbers are in a given group?



Child Tickets

Change History (4)

comment:1 Changed 8 years ago by karsten

Cc: gsathya added

I agree that "*" isn't terribly useful information and that we should replace it with relay counts wherever we can.

While doing so, I wonder if we should merge AS number and AS name columns. Otherwise, it might be unclear whether we're counting unique AS numbers or AS names. What we're really interested in are unique ASes. So, having a single column for them should make things clearer.

We should also put the total count in the fingerprint column, not in the nickname column, because we're counting unique fingerprints, not unique nicknames.

Here's an example (numbers are totally made up):

# CW Advertised Bandwidth P_Guard P_Middle P_Exit Nickname Fingerprint Exit Guard CC AS
1 28.6420% 23.7769% 26.0988% 24.7455% 35.0820% * (567 relays) (182) (251) de (25)
2 21.8611% 27.9476% 25.8289% 21.9512% 17.8038% * (787 relays) (244) (307) us (52)

In the standard, non-grouping case, the AS column would contain something like "AS39138 rrbone UG".


comment:2 Changed 8 years ago by cypherpunks

I like the idea to merge AS number and name. It will help save some space to fit the hostname into the table ;)

comment:3 Changed 8 years ago by gsathya

Status: newneeds_review

Please review bug_6692 branch! Thanks!

comment:4 Changed 8 years ago by karsten

Resolution: implemented
Status: needs_reviewclosed

Looks great! Merged and deployed. Thanks. :)

Note: See TracTickets for help on using tickets.