Changes between Initial Version and Version 1 of Ticket #28565, comment 6


Ignore:
Timestamp:
Feb 8, 2019, 2:42:14 PM (10 months ago)
Author:
juga
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #28565, comment 6

    initial v1  
    11Following the proposed KeyValues in https://pad.riseup.net/p/sbws-exclude-reasons-keep:
    22- any KeyValue that is based on the last bandwidth file adds more code complexity, because it would mean to read and parse the last bandwidth file
    3 - i created 4 additional header KeyValues to report the reasons why the relays were exclued, that would make the KeyValue `recent_measurement_exclusion_count` unneded, since it can be deduced from the previous.
     3- i created 4 additional header KeyValues to report the reasons why the relays were exclued, that would make the KeyValue `recent_measurement_exclusion_count` unneeded, since it can be deduced from the previous.
     4- `relay_recent_measurement_exclusion_count` is 0 all the relays reported, so it'd mean that any relay that gets measurements excluded doesn't make it to be reported.
    45
    56By the initial headers obtained in the Tor network is already explained (or mostly) that sbws is reporting ~1000 relays less than torflow.
    67
     8Edit:
     9 - fix typo
     10 - comment about `relay_recent_measurement_exclusion_count`
    711Branch: https://github.com/juga0/sbws/tree/ticket28565
    812