W3MMD and Desyncs...

[HuanAk]

Can anyone help me identify why my map desyn ?

1, My map never desync if I disable W3MMD system
2, when I enable W3MMD system, very high chance for descyn if player leave
3, chance for descryn is small/none if host have good connection
4, update only call after wave end ( lag is minimum at this time)
5, there are none wait(x second) function happen after wave end ( there is only 1 wait function that do "wait 2 second follow by display message to all player say wave finish)


so I need help to find out why W3MMD have compatibility issue with my code

here is Library code I import:

///////////////////////////////////////////////////////////////
/// The Map Meta Data Library
/// Version: v1.00  	 
/// Last Modified: April 24, 2009
/// Author Chain: Strilanc, [insert next ...]
///////////////////////////////////////////////////////////////
/// This library is used to emit standardized meta data which replay parsers and bot hosts can use to record relevant
/// game statistics like "hero kills" which would otherwise be impossible to record automatically.
///
/// In particular, the flag function can be used to indicate if a leaver should be awarded a win or not. Replays
/// don't contain enough information to easily tell winners who leave from losers who leave. (for example: people
/// who leave while end-game stats are being shown)
///////////////////////////////////////////////////////////////
/// Interface:
///   void FlagPlayer(player, flag_constant)
///   void DefineValue(name, type_constant, goal_constant, suggest_constant)
///   void UpdateValueInt(name, player, operation_constant, value)
///   void UpdateValueReal(name, player, operation_constant, value)
///   void UpdateValueString(name, player, value)
///   void DefineEvent0(name, format)
///   void DefineEvent1(name, format, argName1)
///   void DefineEvent2(name, format, argName1, argName2)
///   void DefineEvent3(name, format, argName1, argName2, argName3)
///   void LogEvent0(name)
///   void LogEvent1(name, arg0)
///   void LogEvent2(name, arg0, arg1)
///   void LogEvent3(name, arg0, arg1, arg2)
///   void LogCustom(unique_identifier, data)
///   void RaiseGuard(reason)
///////////////////////////////////////////////////////////////
/// Notes:
/// - Errors are displayed using BJDebugMsg
/// - Don't try to update a value before defining it
/// - Parsers expect a very specific format, don't screw with the library's output.
/// - If you emit a bunch of data per second, you will cause bandwidth problems for dial-up users. Try to avoid
/// emitting lots of data all at once except at the start and end of games or rounds.
/// - An event's format string uses {#} to represent arguments
/// - Calling RaiseGuard will increase the number of senders for each message from 1 to 3. This increases
/// security but uses more network bandwidth. It is done automatically if tampering is detected.
///////////////////////////////////////////////////////////////
library MMD initializer init
    globals
        public constant integer GOAL_NONE = 101
        public constant integer GOAL_HIGH = 102
        public constant integer GOAL_LOW = 103
        
        public constant integer TYPE_STRING = 101
        public constant integer TYPE_REAL = 102
        public constant integer TYPE_INT = 103

        public constant integer OP_ADD = 101
        public constant integer OP_SUB = 102
        public constant integer OP_SET = 103

        public constant integer SUGGEST_NONE = 101
        public constant integer SUGGEST_TRACK = 102
        public constant integer SUGGEST_LEADERBOARD = 103

        public constant integer FLAG_DRAWER = 101
        public constant integer FLAG_LOSER = 102
        public constant integer FLAG_WINNER = 103
        public constant integer FLAG_LEAVER = 104
        public constant integer FLAG_PRACTICING = 105
    endglobals

    ///////////////////////////////////////////////////////////////
    /// Private variables and constants
    ///////////////////////////////////////////////////////////////
    globals
        private constant boolean SHOW_DEBUG_MESSAGES = true
        
        private constant string chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789_-+= \\!@#$%^&*()/?>.<,;:'\"{}[]|`~"
        private constant integer num_chars = StringLength(chars)
        private string array flags
        private string array goals
        private string array ops
        private string array types
        private string array suggestions
        private boolean initialized = false
                
        private gamecache gc = null
        private constant string ESCAPED_CHARS = " \\"
        
        private constant integer CURRENT_VERSION = 1
        private constant integer MINIMUM_PARSER_VERSION = 1
        private constant string FILENAME = "MMD.Dat"
        private constant string M_KEY_VAL = "val:"
        private constant string M_KEY_CHK = "chk:"
        private constant integer NUM_SENDERS_NAIVE = 1
        private constant integer NUM_SENDERS_SAFE = 3
        private integer num_senders = NUM_SENDERS_NAIVE
        private integer num_msg = 0        
        
        private timer clock = CreateTimer()
        private string array q_msg
        private real array q_time
        private integer array q_index
        private keyword QueueNode
        private QueueNode q_head = 0
        private QueueNode q_tail = 0
    endglobals
    
    ///////////////////////////////////////////////////////////////
    /// Private functions
    ///////////////////////////////////////////////////////////////
    
    ///Triggered when tampering is detected. Increases the number of safeguards against tampering.
    public function RaiseGuard takes string reason returns nothing
        debug if SHOW_DEBUG_MESSAGES then
        debug     call BJDebugMsg("MMD: Guard Raised! (" + reason + ")")
        debug endif
        set num_senders = NUM_SENDERS_SAFE //increase number of players voting on each message
    endfunction

    ///Returns seconds elapsed in game time
    private function time takes nothing returns real
        return TimerGetElapsed(clock)
    endfunction
    
    ///Initializes the char-to-int conversion
    private function prepC2I takes nothing returns nothing
        local integer i = 0
        local string id
        loop
            exitwhen i >= num_chars
            set id = SubString(chars, i, i+1)
            if id == StringCase(id, true) then
                set id = id + "U"
            endif
            call StoreInteger(gc, "c2i", id, i)
            set i = i + 1
        endloop
    endfunction
    ///Converts a character to an integer
    private function C2I takes string c returns integer
        local integer i
        local string id = c
        if id == StringCase(id, true) then
            set id = id + "U"
        endif
        set i = GetStoredInteger(gc, "c2i", id)
        if (i < 0 or i >= num_chars or SubString(chars, i, i+1) != c) and HaveStoredInteger(gc, "c2i", id) then
            //A cheater sent a fake sync to screw with the cached values
            set i = 0
            loop
                exitwhen i >= num_chars //just a weird character
                if c == SubString(chars, i, i+1) then //cheating!
                    call RaiseGuard("c2i poisoned")
                    call StoreInteger(gc, "c2i", id, i)
                    exitwhen true
                endif
                set i = i + 1
            endloop
        endif
        return i
    endfunction

    ///Computes a weak hash value, hopefully secure enough for our purposes
    private function poor_hash takes string s, integer seed returns integer
        local integer n = StringLength(s)
        local integer m = n + seed
        local integer i = 0
        loop
            exitwhen i >= n
            set m = m * 41 + C2I(SubString(s, i, i+1))
            set i = i + 1
        endloop
        return m
    endfunction

    ///Stores previously sent messages for tamper detection purposes
    private struct QueueNode
        readonly real timeout
        readonly string msg
        readonly integer checksum
        readonly string key
        public QueueNode next = 0
        public static method create takes integer id, string msg returns QueueNode
            local QueueNode this = QueueNode.allocate()
            set .timeout = time() + 7.0 + GetRandomReal(0, 2+0.1*GetPlayerId(GetLocalPlayer()))
            set .msg = msg
            set .checksum = poor_hash(.msg, id)
            set .key = I2S(id)
            return this
        endmethod
        private method onDestroy takes nothing returns nothing
            call FlushStoredInteger(gc, M_KEY_VAL+.key, .msg)
            call FlushStoredInteger(gc, M_KEY_CHK+.key, .key)
            set .msg = null
            set .key = null
            set .next = 0
        endmethod
        public method send takes nothing returns nothing
            call StoreInteger(gc, M_KEY_VAL+.key, .msg, .checksum)
            call StoreInteger(gc, M_KEY_CHK+.key, .key, .checksum)
            call SyncStoredInteger(gc, M_KEY_VAL+.key, .msg)
            call SyncStoredInteger(gc, M_KEY_CHK+.key, .key)
        endmethod
    endstruct
    
    ///Returns true for a fixed size uniform random subset of players in the game
    private function isEmitter takes nothing returns boolean
        local integer i = 0
        local integer n = 0
        local integer r
        local integer array picks
        local boolean array pick_flags
        loop
            exitwhen i >= 12
            if GetPlayerController(Player(i)) == MAP_CONTROL_USER and GetPlayerSlotState(Player(i)) == PLAYER_SLOT_STATE_PLAYING then
                if n < num_senders then //initializing picks
                    set picks[n] = i
                    set pick_flags[i] = true
                else //maintain the invariant 'P(being picked) = c/n'
                    set r = GetRandomInt(0, n)
                    if r < num_senders then
                        set pick_flags[picks[r]] = false
                        set picks[r] = i
                        set pick_flags[i] = true
                    endif
                endif
                set n = n + 1
            endif
            set i = i + 1
        endloop
        return pick_flags[GetPlayerId(GetLocalPlayer())]
    endfunction
    
    ///Places meta-data in the replay and in network traffic
    private function emit takes string message returns nothing
        local QueueNode q
        if not initialized then
            call BJDebugMsg("MMD Emit Error: Library not initialized yet.")
            return
        endif
        
        //remember sent messages for tamper check
        set q = QueueNode.create(num_msg, message)
        if q_head == 0 then
            set q_head = q
        else
            set q_tail.next = q
        endif
        set q_tail = q
                
        //send new message
        set num_msg = num_msg + 1
        if isEmitter() then
            call q.send()
        endif
    endfunction

    ///Performs tamper checks
    private function tick takes nothing returns nothing
        local QueueNode q
        local integer i
        
        //check previously sent messages for tampering
        set q = q_head
        loop
            exitwhen q == 0 or q.timeout >= time()
            if not HaveStoredInteger(gc, M_KEY_VAL+q.key, q.msg) then
                call RaiseGuard("message skipping")
                call q.send()
            elseif not HaveStoredInteger(gc, M_KEY_CHK+q.key, q.key) then
                call RaiseGuard("checksum skipping")
                call q.send()
            elseif GetStoredInteger(gc, M_KEY_VAL+q.key, q.msg) != q.checksum then
                call RaiseGuard("message tampering")
                call q.send()
            elseif GetStoredInteger(gc, M_KEY_CHK+q.key, q.key) != q.checksum then
                call RaiseGuard("checksum tampering")
                call q.send()
            endif
            set q_head = q.next
            call q.destroy()
            set q = q_head
        endloop
        if q_head == 0 then
            set q_tail = 0
        endif
        
        //check for future message tampering
        set i = 0
        loop
            exitwhen not HaveStoredInteger(gc, M_KEY_CHK+I2S(num_msg), I2S(num_msg))
            call RaiseGuard("message insertion")
            call emit("Blank")
            set i = i + 1
            exitwhen i >= 10
        endloop
    endfunction
    
    ///Replaces control characters with escape sequences
    private function pack takes string value returns string
        local integer j
        local integer i = 0
        local string result = ""
        local string c
        loop //for each character in argument string
            exitwhen i >= StringLength(value)
            set c = SubString(value, i, i+1)
            set j = 0
            loop //for each character in escaped chars string
                exitwhen j >= StringLength(ESCAPED_CHARS)
                //escape control characters
                if c == SubString(ESCAPED_CHARS, j, j+1) then
                    set c = "\\" + c
                    exitwhen true
                endif
                set j = j + 1
            endloop
            set result = result + c
            set i = i + 1
        endloop
        return result
    endfunction
    
    ///Updates the value of a defined variable for a given player
    private function update_value takes string name, player p, string op, string value, integer val_type returns nothing
        local integer id = GetPlayerId(p)
        if p == null or id < 0 or id >= 12 then
            call BJDebugMsg("MMD Set Error: Invalid player. Must be P1 to P12.")
        elseif val_type != GetStoredInteger(gc, "types", name) then
            call BJDebugMsg("MMD Set Error: Updated value of undefined variable or used value of incorrect type.")
        elseif StringLength(op) == 0 then
            call BJDebugMsg("MMD Set Error: Unrecognized operation type.")        
        elseif StringLength(name) > 50 then
            call BJDebugMsg("MMD Set Error: Variable name is too long.")
        elseif StringLength(name) == 0 then
            call BJDebugMsg("MMD Set Error: Variable name is empty.")
        else
            call emit("VarP " + I2S(id) + " " + pack(name) + " " + op + " " + value)
        endif
    endfunction

    ///Defines an event's arguments and format
    private function DefineEvent takes string name, integer num_args, string format, string arg_data returns nothing
        if GetStoredInteger(gc, "events", name) != 0 then
            call BJDebugMsg("MMD DefEvent Error: Event redefined.")
        else
            call StoreInteger(gc, "events", name, num_args+1)
            call emit("DefEvent " + pack(name) + " " + I2S(num_args) + " " + arg_data + pack(format))
        endif
    endfunction
    
    ///Places an event in the meta-data
    private function LogEvent takes string name, integer num_args, string data returns nothing
        if GetStoredInteger(gc, "events", name) != num_args+1 then
            call BJDebugMsg("MMD LogEvent Error: Event not defined or defined with different # of args.")
        else
            call emit("Event " + pack(name) + data)
        endif
    endfunction

    ///////////////////////////////////////////////////////////////
    /// Public functions
    ///////////////////////////////////////////////////////////////

    ///Sets a player flag like "win_on_leave"
    public function FlagPlayer takes player p, integer flag_type returns nothing
        local string flag = flags[flag_type]
        local integer id = GetPlayerId(p)
        if p == null or id < 0 or id >= 12 then
            call BJDebugMsg("MMD Flag Error: Invalid player. Must be P1 to P12.")
        elseif StringLength(flag) == 0 then
            call BJDebugMsg("MMD Flag Error: Unrecognized flag type.")
        elseif GetPlayerController(Player(id)) == MAP_CONTROL_USER then
            call emit("FlagP " + I2S(id) + " " + flag)
        endif
    endfunction

    ///Defines a variable to store things in
    public function DefineValue takes string name, integer value_type, integer goal_type, integer suggestion_type returns nothing
        local string goal = goals[goal_type]
        local string vtype = types[value_type]
        local string stype = suggestions[suggestion_type]
        if goal == null then
            call BJDebugMsg("MMD Def Error: Unrecognized goal type.")
        elseif vtype == null then
            call BJDebugMsg("MMD Def Error: Unrecognized value type.")
        elseif stype == null then
            call BJDebugMsg("Stats Def Error: Unrecognized suggestion type.")
        elseif StringLength(name) > 32 then
            call BJDebugMsg("MMD Def Error: Variable name is too long.")
        elseif StringLength(name) == 0 then
            call BJDebugMsg("MMD Def Error: Variable name is empty.")
        elseif value_type == TYPE_STRING and goal_type != GOAL_NONE then
            call BJDebugMsg("MMD Def Error: Strings must have goal type of none.")
        elseif GetStoredInteger(gc, "types", name) != 0 then
            call BJDebugMsg("MMD Def Error: Value redefined.")
        else
            call StoreInteger(gc, "types", name, value_type)
            call emit("DefVarP " + pack(name) + " " + vtype + " " + goal + " " + stype)
        endif
    endfunction

    ///Updates the value of an integer variable
    public function UpdateValueInt takes string name, player p, integer op, integer value returns nothing
        call update_value(name, p, ops[op], I2S(value), TYPE_INT)
    endfunction
    
    ///Updates the value of a real variable
    public function UpdateValueReal takes string name, player p, integer op, real value returns nothing
        call update_value(name, p, ops[op], R2S(value), TYPE_REAL)
    endfunction
    
    ///Updates the value of a string variable
    public function UpdateValueString takes string name, player p, string value returns nothing
        local string q = "\""
        call update_value(name, p, ops[OP_SET], q + pack(value) + q, TYPE_STRING)
    endfunction    
    
    public function DefineEvent0 takes string name, string format returns nothing
        call DefineEvent(name, 0, format, "")
    endfunction
    public function DefineEvent1 takes string name, string format, string argName0 returns nothing
        call DefineEvent(name, 1, format, pack(argName0) + " ")
    endfunction
    public function DefineEvent2 takes string name, string format, string argName0, string argName1 returns nothing
        call DefineEvent(name, 2, format, pack(argName0) + " " + pack(argName1) + " ")
    endfunction
    public function DefineEvent3 takes string name, string format, string argName0, string argName1, string argName2 returns nothing
        call DefineEvent(name, 3, format, pack(argName0) + " " + pack(argName1) + " " + pack(argName2) + " ")
    endfunction
    
    public function LogEvent0 takes string name returns nothing
        call LogEvent(name, 0, "")
    endfunction
    public function LogEvent1 takes string name, string arg0 returns nothing
        call LogEvent(name, 1, " " + pack(arg0))
    endfunction
    public function LogEvent2 takes string name, string arg0, string arg1 returns nothing
        call LogEvent(name, 2, " " + pack(arg0) + " " + pack(arg1))
    endfunction
    public function LogEvent3 takes string name, string arg0, string arg1, string arg2 returns nothing
        call LogEvent(name, 3, " " + pack(arg0) + " " + pack(arg1) + " " + pack(arg2))
    endfunction

    ///Emits meta-data which parsers will ignore unless they are customized to understand it
    public function LogCustom takes string unique_identifier, string data returns nothing
        call emit("custom " + pack(unique_identifier) + " " + pack(data))
    endfunction

    ///////////////////////////////////////////////////////////////
    /// Initialization
    ///////////////////////////////////////////////////////////////
    
    ///Emits initialization data
    private function init2 takes nothing returns nothing
        local integer i
        local trigger t
        set initialized = true
        
        call emit("init version " + I2S(MINIMUM_PARSER_VERSION) + " " + I2S(CURRENT_VERSION))

        set i = 0
        loop
            exitwhen i >= 12
            if GetPlayerController(Player(i)) == MAP_CONTROL_USER and GetPlayerSlotState(Player(i)) == PLAYER_SLOT_STATE_PLAYING then
                call emit("init pid " + I2S(i) + " " + pack(GetPlayerName(Player(i))))
            endif
            set i = i + 1
        endloop
        
        set t = CreateTrigger()
        call TriggerAddAction(t, function tick)
        call TriggerRegisterTimerEvent(t, 0.37, true)
    endfunction
    
    ///Places init2 on a timer, initializes game cache, and translates constants
    private function init takes nothing returns nothing
        local trigger t = CreateTrigger()
        call TriggerRegisterTimerEvent(t, 0, false)
        call TriggerAddAction(t, function init2)
        
        set goals[GOAL_NONE] = "none"
        set goals[GOAL_HIGH] = "high"
        set goals[GOAL_LOW] = "low"
        
        set types[TYPE_INT] = "int"
        set types[TYPE_REAL] = "real"
        set types[TYPE_STRING] = "string"

        set suggestions[SUGGEST_NONE] = "none"
        set suggestions[SUGGEST_TRACK] = "track"
        set suggestions[SUGGEST_LEADERBOARD] = "leaderboard"

        set ops[OP_ADD] = "+="
        set ops[OP_SUB] = "-="
        set ops[OP_SET] = "="

        set flags[FLAG_DRAWER] = "drawer"
        set flags[FLAG_LOSER] = "loser"
        set flags[FLAG_WINNER] = "winner"
        set flags[FLAG_LEAVER] = "leaver"
        set flags[FLAG_PRACTICING] = "practicing"

        call FlushGameCache(InitGameCache(FILENAME))
        set gc = InitGameCache(FILENAME)
        call TimerStart(clock, 999999999, false, null)
        call prepC2I()
    endfunction
endlibrary

Here is my code run after game initial (after game mode enter)

// this trigger will run when :
// - "Start Game" trigger

function Trig_W3MMD_SetupInitialValue_Function takes nothing returns nothing
    // player name is constant, not need to update
    call MMD_UpdateValueString( "player_name", GetEnumPlayer(), GetPlayerName(GetEnumPlayer()) )
    call MMD_FlagPlayer(GetEnumPlayer(), 105 ) // FLAG_PRACTICING = 105
    
endfunction

function Trig_W3MMD_DefineValue_Actions takes nothing returns nothing
    // MMD_TYPE_STRING  101 // MMD_TYPE_REAL 102 // MMD_TYPE_INT 103
    // MMD_GOAL_NONE  101 // MMD_GOAL_HIGH 102 // MMD_GOAL_LOW 103
    // MMD_SUGGEST_NONE   101 // MMD_SUGGEST_TRACK  102  // MMD_SUGGEST_LEADERBOARD 103
    
    // name of player
    call MMD_DefineValue( "player_name", 101, 101, 103 )
    // which race
    call MMD_DefineValue( "race", 101, 101, 103 )
    // player's value
    call MMD_DefineValue( "value", 103, 102, 102 )
    // player's income
    call MMD_DefineValue( "income", 103, 102, 102 )
    // player's kill count
    call MMD_DefineValue( "kills", 103, 102, 102 )
    // player's leaked count
    call MMD_DefineValue( "leaked", 103, 101, 101 )
    // player's leak caught from other player
    call MMD_DefineValue( "leak_caught", 103, 101, 101 )
    // player's total gold from all source
    call MMD_DefineValue( "gold_total", 103, 103, 103 )
    // player's total gold from income
    call MMD_DefineValue( "gold_income", 103, 103, 103 )
    // player's total lumber from all source
    call MMD_DefineValue( "lumber_total", 103, 103, 103 )
    // number of wisp
    call MMD_DefineValue( "wisp", 103, 101, 101 )
    // lumberjack level
    call MMD_DefineValue( "lumberjack", 103, 102, 102 )
    // food_used
    call MMD_DefineValue( "food_used", 103, 101, 101 )
    // summons_sent
    call MMD_DefineValue( "summons_sent", 103, 101, 101 )
    // king_upgrade
    call MMD_DefineValue( "king_upgrade", 103, 101, 101 )
    // wave level
    call MMD_DefineValue( "level", 103, 101, 103 )
    // time
    call MMD_DefineValue( "time", 103, 101, 101 )
  
    call ForForce( udg_IngamePlayers_PlayerGroup, function Trig_W3MMD_SetupInitialValue_Function )
    
endfunction

//===========================================================================
function InitTrig_W3MMD_DefineValue takes nothing returns nothing
    set gg_trg_W3MMD_DefineValue = CreateTrigger(  )
    call TriggerAddAction( gg_trg_W3MMD_DefineValue, function Trig_W3MMD_DefineValue_Actions )
endfunction

and this trigger will run at the end of every wave

// this trigger will run when :
// - wave end


function Trig_W3MMD_UpdateValue1_Function takes nothing returns nothing
    local player p = GetEnumPlayer()
    local integer id = GetPlayerId(p) + 1
    
    //OP_SET = 103   FLAG_PRACTICING = 105
    
        
        // player's kill count
        call MMD_UpdateValueInt( "kills", p, 103, udg_PlayerKills_Integer[id] )
        // player's leaked count
        call MMD_UpdateValueInt( "leaked", p, 103, udg_Leaks[id] )
        // player's leak caught
        call MMD_UpdateValueInt( "leak_caught", p, 103, udg_LeaksCaught[id] )
        // player's total gold from all source
        call MMD_UpdateValueInt( "gold_total", p, 103, GetPlayerState(p, PLAYER_STATE_GOLD_GATHERED)  )
        // player's total gold from income
        call MMD_UpdateValueInt( "gold_income", p, 103, udg_GoldFromIncome[id]  )
        // player's total lumber 
        call MMD_UpdateValueInt( "lumber_total", p, 103, GetPlayerState(p, PLAYER_STATE_LUMBER_GATHERED)  )
        // summons_sent
        call MMD_UpdateValueInt( "summons_sent", p, 103, udg_numberOfSummons[id]  )
        // king_upgrade
        call MMD_UpdateValueInt( "king_upgrade", p, 103, udg_UpgradesBought[id]  )  
        // wave level
        call MMD_UpdateValueInt( "level", p, 103, udg_Level_Integer  )  
   
   set p = null

endfunction

function Trig_W3MMD_UpdateValue1_Actions takes nothing returns nothing
    call ForForce( udg_IngamePlayers_PlayerGroup, function Trig_W3MMD_UpdateValue1_Function )
endfunction

//===========================================================================
function InitTrig_W3MMD_UpdateValue_1 takes nothing returns nothing
    set gg_trg_W3MMD_UpdateValue_1 = CreateTrigger(  )
    call TriggerAddAction( gg_trg_W3MMD_UpdateValue_1, function Trig_W3MMD_UpdateValue1_Actions )
endfunction

to run above trigger without wait :

• Countdown Timer - Start W3MMDTimer1 as a One-shot timer that will expire in 5.00 seconds
• Countdown Timer - Start W3MMDTimer2 as a One-shot timer that will expire in 10.00 seconds
• //update only start 5 seconds after wave end, to avoid busy network traffic

• // split update into two part, to reduce network traffic
• W3MMD Timer1 Expired
  • Events
    • Time - W3MMDTimer1 expires
  • Conditions
  • Actions
    • Trigger - Run W3MMD UpdateValue 1 <gen> (ignoring conditions)
• W3MMD Timer2 Expired
  • Events
    • Time - W3MMDTimer2 expires
  • Conditions
  • Actions
    • Trigger - Run W3MMD UpdateValue 2 <gen> (ignoring conditions)

 

[HuanAk]

Bump, add code.

anyone able to help to find out which part of code have compatibility issue ?

there are very high chance of mass descyn if :
- 1 player leave
- then mass descyn in next W3MMD update start.

here is code for player leave "

• Player Leaves
  • Events
    • Player - Player 1 (Red) leaves the game
    • Player - Player 2 (Blue) leaves the game
    • Player - Player 3 (Teal) leaves the game
    • Player - Player 4 (Purple) leaves the game
    • Player - Player 5 (Yellow) leaves the game
    • Player - Player 6 (Orange) leaves the game
    • Player - Player 7 (Green) leaves the game
    • Player - Player 8 (Pink) leaves the game
  • Conditions
  • Actions
    • Player Group - Remove (Triggering player) from IngamePlayers_PlayerGroup

 

[PurgeandFire]

That system involves a lot of local data for each player. It assigns data in variables that end up having different values for each player, so when the variables are read in conditions, the following actions end up being considered local. I am assuming that it ends up Saving/Removing data from the hash locally or that the string table becomes desynced, which ends up causing the disconnects.

As far as fixing it goes, I am unsure. It is kind of a specific system, and only those who know a lot about replays and everything else the system deals with will know how to fix it. I recommend you try to message Strilanc on wc3c.net with your information on the desyncs, or you should just give up the system and opt for something else.

 

[HuanAk]

been reading around, and slowly understand what he did now.

here is something I found useful information may relate to this desyncs:

http://www.wc3jass.com/viewtopic.php?t=2713

//This function takes nothing and returns the host of the game.
function GetHost takes nothing returns player
    //This stores the Id + 1 for each player.
    call StoreInteger(GameCache(), "missionKey", "key", GetPlayerId(GetLocalPlayer()) + 1)
    //Setup the TriggerSyncReady call.
    call TriggerSyncStart()
    //Sync the value of the entry for each player.
    //Each value will sync to the value of the host.
    call SyncStoredInteger(GameCache(), "missionKey", "key")
    //Wait until the Game Cache syncs the key for everyone.
    call TriggerSyncReady()
    //Return the synced value as a player.
    return Player(GetStoredInteger(GameCache(), "missionKey", "key") - 1)
endfunction


I got to do some more tests.. and this seems to have problems when a player leaves, even if it's not the host that left. For some reason, when a player left, the GetHost function returned a different player. I'll look further into it.

still try fix this problem:
- W3MMD not always updated in replay.
- Update "String" is more reliable than "integer" data for unknow reason.
- if any player leave game, major desyncs happen in next value update.

 

[PurgeandFire]

He uses StoreInteger() and GetStoredInteger() to store/retrieve the meta data. SyncStoredInteger() just synchronizes the stored data between the players. (since it is stored locally... it is synced so that it won't have whacky results) I believe he uses integers because it is less memory to hold. He just uses a hashing function to do string-to-integer conversions.

In replays, you can use hostbots or replay parsers (like the one Strilanc posted on wc3c.net) to read the meta data to determine winners, stats, and other things, assuming you used the system correctly. This is only really useful if you want to create your own database to store statistics of players or whatever.

Maybe it is the string table that is becoming desynced. You can try assigning a variable, in map initialization, to each of the strings that he uses. This will ensure that it is entered in the string table for everybody instead of locally.

But you may want to first consider what you are going to use this for. In my opinion, let your map develop first, and just keep this in mind for later when you have finalized your map. You might end up finding that you don't need it. =)

 

[HuanAk]

my desync is temporary fixed, here is some information for anyone who search this post in future :
- Update "String" is more reliable than "integer" data for unknown reason. so try convert to string and update as string
- if any player leave game, major desyncs may happen, because Warcraft have to make sure replay are same on all player.
- desync never happen on hostbot (hostbot can wait or reconnect player to avoid data lost)

so how do I fix this problem ?

I just disable W3MMD define & update.

then turn it on if game is host by Hostbot. (detect by HCL command used)

so this W3MMD desync issue is temporary fixed for anyone who have same problem