Multi Player Instancebility

[UgoUgo]

Hello guys, Im wondering if my trigger below is MPI.

Triggers

• CRoom Port 6 TOP
  • Events
    • Unit - A unit enters CloneRoomPort6TOP <gen>
  • Conditions
    • CRoomWarping[(Player number of (Owner of (Triggering unit)))] Equal to False
  • Actions
    • Set CRoomWarping[(Player number of (Owner of (Triggering unit)))] = True
    • Set CRoomPoint[(Player number of (Owner of (Triggering unit)))] = (Center of CloneRoomPort1BTM <gen>)
    • Set CRoomPoint2[(Player number of (Owner of (Triggering unit)))] = (Center of CRoom1EntBtm <gen>)
    • Set CRoomPoint3[(Player number of (Owner of (Triggering unit)))] = (Position of (Triggering unit))
    • Set CRoomPoint4[(Player number of (Owner of (Triggering unit)))] = (CRoomPoint3[(Player number of (Owner of (Triggering unit)))] offset by 600.00 towards 90.00 degrees)
    • Unit - Make (Triggering unit) Invulnerable
    • Unit - Turn collision for (Triggering unit) Off
    • Unit - Order (Triggering unit) to Move To CRoomPoint4[(Player number of (Owner of (Triggering unit)))]
    • Unit - Set (Triggering unit) movement speed to 300.00
    • Set CRoomFilter = 0.00
    • Custom script: if GetLocalPlayer() != GetOwningPlayer(GetTriggerUnit()) then
    • Set CRoomFilter = 100.00
    • Custom script: endif
    • Cinematic - Fade out over 0.70 seconds using texture Black Mask and color (0.00%, 0.00%, 0.00%) with CRoomFilter% transparency
    • Wait 1.00 seconds
    • Unit - Set (Triggering unit) movement speed to (Default movement speed of (Triggering unit))
    • Camera - Set the camera bounds for (Owner of (Triggering unit)) to CRoom1 <gen>
    • Unit - Move (Triggering unit) instantly to CRoomPoint[(Player number of (Owner of (Triggering unit)))]
    • Unit - Order (Triggering unit) to Move To CRoomPoint2[(Player number of (Owner of (Triggering unit)))]
    • Camera - Pan camera for (Owner of (Triggering unit)) to CRoomPoint[(Player number of (Owner of (Triggering unit)))] over 0.00 seconds
    • Set CRoomFilter = 0.00
    • Custom script: if GetLocalPlayer() != GetOwningPlayer(GetTriggerUnit()) then
    • Set CRoomFilter = 100.00
    • Custom script: endif
    • Cinematic - Fade in over 0.70 seconds using texture Black Mask and color (0.00%, 0.00%, 0.00%) with CRoomFilter% transparency
    • Camera - Lock camera target for (Owner of (Triggering unit)) to (Triggering unit), offset by (0.00, 0.00) using Default rotation
    • Wait 0.30 seconds
    • Set CRoomWarping[(Player number of (Owner of (Triggering unit)))] = False
    • Unit - Turn collision for (Triggering unit) On
    • Unit - Make (Triggering unit) Vulnerable
    • Selection - Select (Triggering unit) for (Owner of (Triggering unit))
    • Custom script: call RemoveLocation(udg_CRoomPoint[GetConvertedPlayerId(GetOwningPlayer(GetTriggerUnit()))])
    • Custom script: call RemoveLocation(udg_CRoomPoint2[GetConvertedPlayerId(GetOwningPlayer(GetTriggerUnit()))])
    • Custom script: call RemoveLocation(udg_CRoomPoint3[GetConvertedPlayerId(GetOwningPlayer(GetTriggerUnit()))])
    • Custom script: call RemoveLocation(udg_CRoomPoint4[GetConvertedPlayerId(GetOwningPlayer(GetTriggerUnit()))])

Now there's only 1 unit per player; so I just have to make sure it works for everyone, not every unit.

Basically, Im trying to create a warp system. You know, the ones you see in old games, where when you switch places, you can still see the character moving while the screen is fading out.

 

[deathismyfriend]

it looks mpi to me cant see anything unless i missed it. not sure but y do u set a variable to 100 in the getlocalplayer block ? i think that may cause desyncs

 

[UgoUgo]

I followed Purgeandfire's tutorial for the getlocalplayer, it looked odd to me at first but it seem to make sense as I read his elaboration. I tried some debugging, doesnt seem to desync. But I'll have to confirm it with multiplayer tests.

 

[deathismyfriend]

ok it just doesnt seem needed to me when looking at how ur trigger works.

 

[UgoUgo]

I need the black mask effect ONLY for the owner of unit whoever entered region, something like old ps games.

 

[deathismyfriend]

ooo oops lol i didnt see that variable being used in the filter lol. gotta love being tired. i see what u mean now.

 

[Bannar]

Improve the efficiency of your script with variables usage. You use multiple references to:
- (Player number of (Owner of (Triggering unit)))
- (Owner of (Triggering unit))
- (Triggering unit)

I suggest setting additional variables in following order:

• Set unit = (Triggering unit)
• Set player = (Owner of unit)
• Set index = (Player number of player)

Lets look at this fragment for example:

• Wait 1.00 seconds
• Unit - Set (Triggering unit) movement speed to (Default movement speed of (Triggering unit))

Since your event speaks of unit entering region, if another unit triggers this event during the 1.00 period the reference to (Triggering unit) will now point at newest unit and data about previous one will be lost. You need to fix those issues for trigger to be MUI.

 

[UgoUgo]

Huh, I thought Triggering Unit is a local variable? And I tested with 2 players entering the region with about 0 - 0.2 sec interval, seems to be working fine.

 

[deathismyfriend]

Triggering unit is a local variable and it is still mui or mpi w waits.

He also only wants it to be mpi. Setting the variables would be nice but u would lose mpi that way unless u make those variables local in the trigger but then u need to do more work. Although if this event happens a lot the efficiency gain might be worth the extra work.