Page MenuHomeWildfire Games

Enable Late-Observers by default
ClosedPublic

Authored by elexis on Mar 12 2017, 11:04 PM.

Details

Summary

The Late Observer setting allows users to join a game as observer after it started.
Spectating games is highly interesting, because one can follow the course of actions of selected players,
compare how different strategies and tactics work out, helping to understanding the game mechanics,
makes matches and the players in it more popular and thus 0 A.D. as a whole benefits too.

Besides having a useful feature enabled by default, it also makes it more consistent, because
currently 32 observers can join a game after 8 players joined (independent of the late observer flag)
(as we can't know in advance which clients will become assigned players).

The downsides to this features are players who are not wanted (either due to their personality or due to lag) joining games.
This however is countered by the /kick, /ban, /kickspecs and /banspecs commands. A network dialog to have buttons for this is planned as well.
Some hosts really want this feature to be disabled (as they might not host and moderate as an observer but also play while hosting).
They can still disable the option as usual in the settings.

Test Plan

Test whether the value conforms with your opinion.

Diff Detail

Repository
rP 0 A.D. Public Repository
Branch
/ps/trunk
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 748
Build 1187: Vulcan BuildJenkins
Build 1186: arc lint + arc unit

Event Timeline

elexis created this revision.Mar 12 2017, 11:04 PM
scythetwirler accepted this revision.Mar 12 2017, 11:24 PM
This revision is now accepted and ready to land.Mar 12 2017, 11:24 PM
Vulcan added a subscriber: Vulcan.Mar 12 2017, 11:48 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (302 tests)..............................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (302 tests)..............................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/510/ for more details.

echotangoecho accepted this revision.Mar 15 2017, 1:45 PM
Imarok accepted this revision.Mar 15 2017, 7:20 PM
ffffffff accepted this revision.Mar 16 2017, 4:32 AM
fatherbushido resigned from this revision.Mar 19 2017, 2:10 PM

You know.

Hannibal_Barca accepted this revision.Mar 19 2017, 8:08 PM
bb accepted this revision.Mar 19 2017, 8:41 PM
elexis edited the test plan for this revision. (Show Details)Mar 19 2017, 11:53 PM
This revision was automatically updated to reflect the committed changes.