Ticket #1384 (closed defect: fixed)

Opened 13 months ago

Last modified 13 months ago

Out of sync error

Reported by: historic_bruno Owned by: quantumstate
Priority: Release Blocker Milestone: Alpha 10
Component: UI & Simulation Keywords:
Cc:

Description

Get this OOS error with r11758, on the default map, 2 human players (no AIs), after tasking one of player 1's units to move. Logs are attached, a diff tool shows the only difference to be some extra data in the host's tech manager modificationsCache for player 1:

    },
    "Attack/Melee/Hack": {
      "1170": 6,
      "1461": 15
    },
    "Attack/Melee/Pierce": {
      "1170": 0,
      "1461": 0
    },
    "Attack/Melee/Crush": {
      "1170": 0,
      "1461": 0
    },
    "Armour/Hack": {
      "1170": 1,
      "1461": 9
    },
    "Armour/Pierce": {
      "1170": 1,
      "1461": 8
    },
    "Armour/Crush": {
      "1170": 5,
      "1461": 9
    },
    "ResourceGatherer/BaseSpeed": {
      "1170": 1,
      "1461": 1
    },
    "ResourceGatherer/Rates/food.fish": {
      "1170": 0.5
    },
    "ResourceGatherer/Rates/food.fruit": {
      "1170": 2
    },
    "ResourceGatherer/Rates/food.grain": {
      "1170": 0.4
    },
    "ResourceGatherer/Rates/food.meat": {
      "1170": 1
    },
    "ResourceGatherer/Rates/metal.ore": {
      "1170": 0.5
    },
    "ResourceGatherer/Rates/stone.rock": {
      "1170": 0.5
    },
    "ResourceGatherer/Rates/stone.ruins": {
      "1170": 5
    },
    "ResourceGatherer/Rates/treasure": {
      "1170": 1,
      "1461": 1
    },
    "ResourceGatherer/Rates/wood.ruins": {
      "1170": 5
    },
    "ResourceGatherer/Rates/wood.tree": {
      "1170": 0.5

I suspect GuiInterface.GetEntityState() is indirectly triggering some unsynchronized calls to ApplyModifications(). Is there a way to simply not synchronize the cache? (And a getter should probably never modify the simulation state as it's unexpected behavior)

Attachments

oos_dumps.zip (204.4 KB) - added by historic_bruno 13 months ago.
oos_logs_vts.zip (266.1 KB) - added by vts 13 months ago.

Change History

Changed 13 months ago by historic_bruno

comment:1 Changed 13 months ago by vts

Can reproduce this with the same revision, both running of windows machines (Vista and Win7). Game went out of sync after about ~5 seconds of playtime.

Changed 13 months ago by vts

comment:2 Changed 13 months ago by quantumstate

  • Owner set to quantumstate
  • Status changed from new to closed
  • Resolution set to fixed

In 11761:

Don't serialize the technology modifications cache. Fixes #1384.

Note: See TracTickets for help on using tickets.