paritytech / substrate-debug-kit

A collection of debug tools, scripts and libraries on top of substrate.
GNU General Public License v3.0
95 stars 36 forks source link

Sorting of winners #3

Closed ilhanu closed 4 years ago

ilhanu commented 4 years ago

I changed the list of validators to 100, just to see how far I am from validating. The current sorting of the stake_total doesn't make sense. nr.49 should be higher then nr. 48 and there are a couple of more examples.

#48 == 6e510928e484fe036e6cbfd68ca50b02a79639bdb5282ced5fbce99dfda89c5b (5EZMCPQe...)
  [stake_total: 1813_KSM (1,813,000,000,000,000)] [stake_own: 0_KSM (500,000,000,000) (0%)] [other_stake_sum: 1812_KSM (1,812,500,000,000,000) (99%)] [other_stake_count: 1] [ctrl: 6e510928e484fe036e6cbfd68ca50b02a79639bdb5282ced5fbce99dfda89c5b (5EZMCPQe...)]
  Voters:
    #0 [amount = 1812_KSM (1,812,500,000,000,000)] (68d6825c64c27c28af460e8fad313bb2fed75aff59e8df99b3edcaedec96d866 (5ESAZR9X...), 1812500000000000)
#49 == a8ca45025561ad23d3000cef010fb14901ba322283898e08c3d0a7fcdd03ce5a (5Ft1zHEv...)
  [stake_total: 4270_KSM (4,270,822,532,000,000)] [stake_own: 195_KSM (195,000,000,000,000) (4%)] [other_stake_sum: 4075_KSM (4,075,822,532,000,000) (95%)] [other_stake_count: 7] [ctrl: a8ca45025561ad23d3000cef010fb14901ba322283898e08c3d0a7fcdd03ce5a (5Ft1zHEv...)]
  Voters:
    #0 [amount = 63_KSM (63,315,000,000,000)] (dd24c212f887efc72fc27674e93551da7b81b66c9a2f53926b7bab15c51acac4 (5H4fMnKj...), 63315000000000)
    #1 [amount = 1495_KSM (1,495,843,755,000,000)] (56fdb6ef85bb41e02e3c0586c0a3ffaeed1dfc7b7fddc7d31c1bb79fec14f327 (5E2mLvDT...), 1495843755000000)
    #2 [amount = 1055_KSM (1,055,258,755,000,000)] (8e13995e41dedfa77376e7c67e60b9750f40cda4b27846a29fe3b5715d5af4d7 (5FGzUjPk...), 1055258755000000)
    #3 [amount = 37_KSM (37,896,756,000,000)] (ba9789e37ed91b3bea19b0aadb6dd966ecc12d337794f23dee8694b82bf6163a (5GHMnCUi...), 37896756000000)
    #4 [amount = 211_KSM (211,103,756,000,000)] (f628bc11ec994800136a6fb4567f4fafa26362416a5d42dcf23eb2b4dfc06510 (5HdTjjFr...), 211103756000000)
    #5 [amount = 780_KSM (780,065,755,000,000)] (7c134872c9fc375ef3fc3d54deb576f32738b26595476d21e5c7892f9e611e3e (5EsPXJ12...), 780065755000000)
    #6 [amount = 432_KSM (432,338,755,000,000)] (56208625541b6ecf1bcc521bffdf173a21433afacfae5fccb3bbed79840f8905 (5E1ddtmA...), 432338755000000)
#50 == 90ea97930be1db7ea6b5d19d84f7da70e24b93fa3305dcc4e821b5cbc7d6c460 (5FLiSXf4...)
  [stake_total: 1813_KSM (1,813,000,000,000,000)] [stake_own: 0_KSM (500,000,000,000) (0%)] [other_stake_sum: 1812_KSM (1,812,500,000,000,000) (99%)] [other_stake_count: 1] [ctrl: 90ea97930be1db7ea6b5d19d84f7da70e24b93fa3305dcc4e821b5cbc7d6c460 (5FLiSXf4...)]
  Voters:
    #0 [amount = 1812_KSM (1,812,500,000,000,000)] (68d6825c64c27c28af460e8fad313bb2fed75aff59e8df99b3edcaedec96d866 (5ESAZR9X...), 1812500000000000)
#51 == bc1deacfc7e5c6e5f0373560c14fbce156ff2a0ed7e208d049ccd985dec85545 (5GKMkGMN...)
  [stake_total: 3621_KSM (3,621,622,938,750,000)] [stake_own: 10_KSM (10,000,000,000,000) (0%)] [other_stake_sum: 3611_KSM (3,611,622,938,750,000) (99%)] [other_stake_count: 1] [ctrl: c08d5de7a5d97bea2c7ddf516d0635bddc43f326ae2f80e2595b49d4a08c4619 (5GRB3c82...)]
  Voters:
    #0 [amount = 3611_KSM (3,611,622,938,750,000)] (8687495febea21de449ff27381f9308da6f15ce5d9d3146261275dec18cd566f (5F76TmCY...), 3611622938750000)
#52 == e03a5b41f85e554baa0b674c2db15db2b1b3c63776b8fa5c461e2dda1202823c (5H8hvFC6...)
  [stake_total: 1813_KSM (1,813,000,000,000,000)] [stake_own: 0_KSM (500,000,000,000) (0%)] [other_stake_sum: 1812_KSM (1,812,500,000,000,000) (99%)] [other_stake_count: 1] [ctrl: e03a5b41f85e554baa0b674c2db15db2b1b3c63776b8fa5c461e2dda1202823c (5H8hvFC6...)]
  Voters:
    #0 [amount = 1812_KSM (1,812,500,000,000,000)] (68d6825c64c27c28af460e8fad313bb2fed75aff59e8df99b3edcaedec96d866 (5ESAZR9X...), 1812500000000000)
soc1c commented 4 years ago

yeah, that's something I couldn't wrap my head around yet.

kianenigma commented 4 years ago

nr.49 should be higher then nr. 48

why? maybe because:

The current sorting of the stake_total

This list os not sorted based on total_stake.