Nividica / FactorioStorageEnergistics

MIT License
3 stars 2 forks source link

Stress Tests / Performance #2

Open Nividica opened 6 years ago

Nividica commented 6 years ago

Reported Low UPS with very high throughput

Nividica commented 6 years ago

Found issue with GUI manager. Corrected issue, but sheds light on larger issue: Accessing the game object is expensive, specifically getting a player. This should be avoided as best it can be. (Now I understand why events give a player_index and not a player)

OskarPepryk commented 6 years ago

I think the performance hit is proportional to the number of requests or number of item types in network. See my screenshots: https://imgur.com/a/t5tot2X 2 first are from testing map, 3rd is from my modded gameplay, where I put everything in the network of a 30 SPM base.