mobs_redo is generating unnecessary network traffic by by calling object:set_properties without need #6150
Labels
No Label
1. kind/balancing
1. kind/breaking
1. kind/bug
1. kind/construction
1. kind/documentation
1. kind/enhancement
1. kind/griefing
1. kind/invalid
1. kind/meme
1. kind/node limit
1. kind/other
1. kind/protocol
2. prio/controversial
2. prio/critical
2. prio/elevated
2. prio/good first issue
2. prio/interesting
2. prio/low
3. source/art
3. source/client
3. source/engine
3. source/ingame
3. source/integration
3. source/lag
3. source/license
3. source/mod upstream
3. source/unknown
3. source/website
4. step/approved
4. step/at work
4. step/blocked
4. step/discussion
4. step/help wanted
4. step/needs confirmation
4. step/partially fixed
4. step/question
4. step/ready to deploy
4. step/ready to QA test
4. step/want approval
5. result/cannot reproduce
5. result/duplicate
5. result/fixed
5. result/maybe
5. result/wontfix
ugh/petz
ugh/QA main
ugh/QA NOK
ugh/QA OK
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: your-land/bugtracker#6150
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Here's some previous discussion:
#5837 (comment)
Patch provided there also:
#5837 (comment)
Should I just push this fix toyour-land
? It should benefit when there are lots of mobs...upstream pr:
https://codeberg.org/tenplus1/mobs_redo/pulls/11
Fixed upstream here:
70118fc8da
Great! Thank you for the PR and getting it merged! That'll help in Voice battles as well. It is most likely not the only issue that made such fights more...packeet-intensive in the past, but it will certainly reduce traffic. And that's very good.
The fix is already on the testserver :)
Checked with wireshark again: don't see mesh and texture names spam anymore... What's left is I assume just
set_rotation
for "looking around" effect, but those messages are much smaller and can fit 6 into one network packet at least...(and nametag color change that depends on HP seems to still work too)
Sounds like QA/OK?
Ok, label added :p
assuming this is live