MantisBT - Zandronum
View Issue Details
0004090Zandronum[All Projects] Suggestionpublic2023-01-20 05:302024-01-10 03:14
WaTaKiD 
DrinkyBird 
normalfeatureN/A
resolvedfixed 
MicrosoftWindowsXP/Vista/7
3.1 
3.23.2 
0004090: improve the dumptrafficmeasure ccmd's sorting
dumptrafficmeasure is a handy tool for finding and optimizing bandwidth hungry actors/scripts

said tool could itself be optimized by listing actors/scripts by their traffic, as opposed to having to scour a potentially long list of scrambled entries, we could instead just glance at the first (or last?) few entries
other ideas from discord:

Trillster: ascending order may be more ideal for console, given it'll only show the most recent lines if you've got a bunch of different actors in play
Trillster: so the highest count actors would always show at the bottom of the list

Kaminsky: There could be an optional parameter to order entries either in ascending or descending order.

there was also a bit about color coding the traffic values depending on size but i feel thatll get complicated, and not all consoles display color
No tags attached.
Issue History
2023-01-20 05:30WaTaKiDNew Issue
2023-01-20 05:37WaTaKiDNote Added: 0022706
2023-01-20 05:45KaminskyNote Added: 0022707
2023-01-20 05:45KaminskyTarget Version => 3.2
2023-03-19 21:39DrinkyBirdNote Added: 0022813
2023-03-19 21:51DrinkyBirdAssigned To => DrinkyBird
2023-03-19 21:51DrinkyBirdStatusnew => needs testing
2024-01-02 08:11Ru5tK1ngNote Added: 0022970
2024-01-10 03:14Ru5tK1ngResolutionopen => fixed
2024-01-10 03:14Ru5tK1ngFixed in Version => 3.2
2024-01-10 03:14Ru5tK1ngStatusneeds testing => resolved

Notes
(0022706)
WaTaKiD   
2023-01-20 05:37   
Kaminsky: Something else worth mentioning that wasn't discussed here or in the ticket: if there's no actor or script traffic dump entries, then it shouldn't print the "Network traffic (in bytes) caused by actor code pointers:" or "Network traffic (in bytes) caused by ACS scripts:" messages.

Kaminsky: If there's no traffic dump at all, then it should probably print something like "there's no network traffic dump available".
(0022707)
Kaminsky   
2023-01-20 05:45   
Samuzero15tlh: could be possible to add the number of actors that are causing the traffic delays?

Perhaps this could also be useful information to include in the traffic dump?
(0022813)
DrinkyBird   
2023-03-19 21:39   
'https://hg.osdn.net/view/zandronum/zandronum-stable/rev/b24867c9a99e [^]'
(0022970)
Ru5tK1ng   
2024-01-02 08:11   
Tested with r231220. dumptrafficmeasure worked as described in the OSDN changelog. Adding the desc parameter sorted the lists from largest to smallest.

Is the amount of actors going to be added to the traffic dump? If not then this issue is pretty much done.