Skip to content
This repository has been archived by the owner on Jun 25, 2021. It is now read-only.

Estimate network size as per @Thierry61's suggestion #1358

Merged
merged 2 commits into from
Feb 15, 2017

Conversation

michaelsproul
Copy link
Contributor

Updated version of #1353

@maidsafe-highfive
Copy link

r? @afck

(maidsafe_highfive has picked a reviewer for you, use r? to override)

@michaelsproul
Copy link
Contributor Author

Here's what it looks like in the perfect and imperfect information cases respectively:

INFO 07:02:49.244218000 [routing::states::node node.rs:281]  ---------------------------------------------------------- 
INFO 07:02:49.244270000 [routing::states::node node.rs:282] | Node(55a5f3..(010)) PeerId(19) - Routing Table size:  43 |
INFO 07:02:49.244296000 [routing::states::node node.rs:283] | Exact network size: 44                                   |
INFO 07:02:49.244328000 [routing::states::node node.rs:284]  ---------------------------------------------------------- 
INFO 07:02:48.592343000 [routing::states::node node.rs:281]  ---------------------------------------------------------- 
INFO 07:02:48.592373000 [routing::states::node node.rs:282] | Node(223bf6..(001)) PeerId(11) - Routing Table size:  36 |
INFO 07:02:48.592397000 [routing::states::node node.rs:283] | Estimated network size: 43                               |
INFO 07:02:48.592428000 [routing::states::node node.rs:284]  ---------------------------------------------------------- 

@afck afck merged commit f787f1f into maidsafe:master Feb 15, 2017
@michaelsproul michaelsproul deleted the net-size-estimate branch February 15, 2017 09:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants