Zandronum Chat on our Discord Server Get the latest version: 3.1
Source Code
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0004090Zandronum[All Projects] Suggestionpublic2023-01-20 05:302024-01-10 03:14
ReporterWaTaKiD 
Assigned ToDrinkyBird 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusresolvedResolutionfixed 
PlatformMicrosoftOSWindowsOS VersionXP/Vista/7
Product Version3.1 
Target Version3.2Fixed in Version3.2 
Summary0004090: improve the dumptrafficmeasure ccmd's sorting
Descriptiondumptrafficmeasure 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
Additional Informationother 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
Attached Files

- Relationships

-  Notes
User avatar (0022706)
WaTaKiD (updater)
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".
User avatar (0022707)
Kaminsky (developer)
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?
User avatar (0022813)
DrinkyBird (developer)
2023-03-19 21:39

 'https://hg.osdn.net/view/zandronum/zandronum-stable/rev/b24867c9a99e [^]'
User avatar (0022970)
Ru5tK1ng (updater)
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.

Issue Community Support
This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.
Supporters: WaTaKiD DrinkyBird
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2023-01-20 05:30 WaTaKiD New Issue
2023-01-20 05:37 WaTaKiD Note Added: 0022706
2023-01-20 05:45 Kaminsky Note Added: 0022707
2023-01-20 05:45 Kaminsky Target Version => 3.2
2023-03-19 21:39 DrinkyBird Note Added: 0022813
2023-03-19 21:51 DrinkyBird Assigned To => DrinkyBird
2023-03-19 21:51 DrinkyBird Status new => needs testing
2024-01-02 08:11 Ru5tK1ng Note Added: 0022970
2024-01-10 03:14 Ru5tK1ng Resolution open => fixed
2024-01-10 03:14 Ru5tK1ng Fixed in Version => 3.2
2024-01-10 03:14 Ru5tK1ng Status needs testing => resolved




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2024 MantisBT Team
Powered by Mantis Bugtracker