year6b7a / item-network-factorio-mod

MIT License
3 stars 6 forks source link

Test on multiplayer #5

Open year6b7a opened 1 year ago

year6b7a commented 1 year ago

While nothing jumps out to me as being buggy with multiplayer it's worth checking to make sure.

bengardner commented 1 year ago

There is only one item network. If there is a multiplayer game that is not co-op and there are multiple competing forces, then they would share the item network. Probably not desired. I've not played a game with multiple competing teams. Not sure which mod would enable that.

year6b7a commented 1 year ago

This is intended for multiplayer games where everyone is on the same team. This is mainly to test that UI state is correctly stored per-player rather than globally.

bengardner commented 1 year ago

I finally got around to some LAN play and noticed a really annoying behavior with the usual network latency.

When you click on a chest, the "normal" chest dialog pops up and is displayed for a few ticks before it is replaced with the custom dialog.

That obviously doesn't happen on a single player game where there is no latency. I have no idea what to do about that. We'd have to find a way to suppress the original chest UI.

Good news is that I didn't hit any desyncs.

year6b7a commented 1 year ago

Hmm that's super strange. I'm not doing anything fancy with the GUI code so curious if other mods run into this issue as well. If there's an easy fix I'd be happy to merge it in.