Change Sync Response

Change Sync Response

Discuss the source code and development of Spring Engine in general from a technical point of view. Patches go here too.

Moderator: Moderators

Post Reply
User avatar
AF
AI Developer
Posts: 20687
Joined: 14 Sep 2004, 11:32

Change Sync Response

Post by AF »

No sync response is misleading, and should be renamed, users think it's a desync error message when it's not
User avatar
lurker
Posts: 3842
Joined: 08 Jan 2007, 06:13

Re: Dedicated Client

Post by lurker »

My opinion on that depends on one thing. What percent of the confused are native English speakers?
User avatar
Beherith
Posts: 5145
Joined: 26 Oct 2007, 16:21

Re: Dedicated Client

Post by Beherith »

Lobby says 70%+ of players are non native speakers.
User avatar
AF
AI Developer
Posts: 20687
Joined: 14 Sep 2004, 11:32

Re: Dedicated Client

Post by AF »

I see people with US and UK flags get confused, its too easy to misread as a sync error.
SirMaverick
Posts: 834
Joined: 19 May 2009, 21:10

Re: Dedicated Client

Post by SirMaverick »

AF wrote:No sync response is misleading, and should be renamed, users think it's a desync error message when it's not
Any suggestions?
Any message that is not displayed in normal ("all is good") situations can mislead someone into thinking it's an error.
I'd prefer to teach players that if it's not an error its just a warning/hint.
User avatar
AF
AI Developer
Posts: 20687
Joined: 14 Sep 2004, 11:32

Re: Dedicated Client

Post by AF »

warning: delayed response from player XYZ
User avatar
very_bad_soldier
Posts: 1397
Joined: 20 Feb 2007, 01:10

Re: Dedicated Client

Post by very_bad_soldier »

SirMaverick wrote: I'd prefer to teach players that if it's not an error its just a warning/hint.
:shock:
warning: delayed response from player XYZ
much better than the original!
tombom
Posts: 1933
Joined: 18 Dec 2005, 20:21

Re: Change Sync Response

Post by tombom »

"Player X is lagging"
User avatar
Argh
Posts: 10920
Joined: 21 Feb 2005, 03:38

Re: Change Sync Response

Post by Argh »

"Slow response from Player X" perhaps?
SirMaverick
Posts: 834
Joined: 19 May 2009, 21:10

Re: Change Sync Response

Post by SirMaverick »

So it's the word "sync" that is misleading?
User avatar
AF
AI Developer
Posts: 20687
Joined: 14 Sep 2004, 11:32

Re: Change Sync Response

Post by AF »

sync is a highly precise meaning of the test thats used, but its caused by lag or a slow response, and that is what it useful to the end user.

Also when playing the game you sometimes only watch the word sync and assume its a desync, afterall this si an RTS not a turn based game.
User avatar
lurker
Posts: 3842
Joined: 08 Jan 2007, 06:13

Re: Dedicated Client

Post by lurker »

AF wrote:I see people with US and UK flags get confused, its too easy to misread as a sync error.
Well insults to them.

AF wrote:warning: delayed response from player XYZ
No.
very_bad_soldier wrote:
warning: delayed response from player XYZ
much better than the original!
No.
tombom wrote:"Player X is lagging"
No.
Argh wrote:"Slow response from Player X" perhaps?
No.


I'm boggled at these ideas. There is already a 'delayed response' message. This is about the message that says no response. A suitable replacement has to have the meaning of "Player X is not responding". Wrong is worse than confusing.

Side note, maybe Spring shouldn't whine about missing up-to-date sync messages when it's still getting delayed sync messages.
User avatar
CarRepairer
Cursed Zero-K Developer
Posts: 3359
Joined: 07 Nov 2007, 21:48

Re: Change Sync Response

Post by CarRepairer »

Warning: No sync response.
Warning: Delayed sync response.
Error: Player desynced.
User avatar
hoijui
Former Engine Dev
Posts: 4344
Joined: 22 Sep 2007, 09:51

Re: Change Sync Response

Post by hoijui »

what if we introduced log levels for infolog?
Auswaschbar
Spring Developer
Posts: 1254
Joined: 24 Jun 2007, 08:34

Re: Change Sync Response

Post by Auswaschbar »

hoijui wrote:what if we introduced log levels for infolog?
What if we already had them, and they were removed?
User avatar
Argh
Posts: 10920
Joined: 21 Feb 2005, 03:38

Re: Change Sync Response

Post by Argh »

The only things I'd honestly like to see in logging that I can't right now is:

1. I'd like to see debugging logs for Lua be possibly in a different file. It's a pain sometimes, hand-parsing through Infolog, looking for bug events.

2. I'd like to see cyclic logging of the performance metrics, using a parameter to determine how often we see it, or a trigger based on a param. For example: CPU is stalled? Let a Lua callin know, so that we can explore the game-state (which will, obviously, cause yet more lag, but this is for optimization purposes) if we really want that much detail.

I don't see much point in specific logging for AI: they can print their log via either infolog or use Lua (in which case, a seperate Lua debugging log would be doubly useful).
User avatar
hoijui
Former Engine Dev
Posts: 4344
Joined: 22 Sep 2007, 09:51

Re: Change Sync Response

Post by hoijui »

1. use sed, grep or awk
2. there is a chat command: /debuginfo profiling; which loggs the info you see when pressing B to infolog, so i gues it shoudl be easy for you to do a widget and issue that command whenever you need it
(3.) all AIs have their own log, and AI Devs wont change that. languages like Java have nice logging systems, and as AIs are relatively independent parts, and it is easier for the devs to implement their own stuff...
they can log to infolog, but they hardly use it.
Last edited by hoijui on 13 Nov 2009, 10:13, edited 1 time in total.
User avatar
Argh
Posts: 10920
Joined: 21 Feb 2005, 03:38

Re: Change Sync Response

Post by Argh »

I didn't know about that console command. I guess I could monitor for delays ala the performance Widgets we have, then trigger.

I use grep, but not for that purpose. That sounds like it could save me some time, thanks.
User avatar
lurker
Posts: 3842
Joined: 08 Jan 2007, 06:13

Re: Change Sync Response

Post by lurker »

log=io.open("lualog.txt"), then use log:write instead of Echo? What do you want for lua that you can't already do? I admit that trapping all errors in callins is more complex, but if that's what you want I can make a debugging shim to do it in a few minutes.
Post Reply

Return to “Engine”