FakeFishGames / Barotrauma

A 2D online multiplayer game taking place in a submarine travelling through the icy depths of Jupiter's moon Europa.
http://www.barotraumagame.com/
1.68k stars 393 forks source link

Circuit box issues - Update Performance, Corruption, Inconsistencies #13499

Open JimmySlickD opened 4 months ago

JimmySlickD commented 4 months ago

Disclaimers

What happened?

After using the Circuit box i found some issues in the current stable.

  1. Performance Integrating a lot of logic into the circuit box results in severe performance degradation compared to normal wiring - after just 4 boxes with 50+ components and 80+ wires in each, removing the same amount outside the box - Update ms increased from around 4-5ms to 7-8ms. Using boxes would render some otherwise fine wiring unplayable due to its increased performance impact. (tested in editor and multiplayer / multiple players)

  2. Power It really makes no sense that the Circuit box cannot handle power, as this would be useful for a lot of wiring. It also creates inconsistency as the relay component still have the power input inside.

  3. Corrupt components If you mark all components and alter Ability to edit in game check box, it appears to not set the checkbox (and I am unsure whether this feature works) but more importantly some/few components becomes corrupt and must be replaced to gain access to their edit box again. Seems to impact signal and Wifi components. (only tested in editor)

Reproduction steps

Tested with these files: Original wiring: Riftpiercer X97-S1.zip With converted to Circuit box wiring: Riftpiercer X98-S1.zip

Bug prevalence

Happens every time I play

Single player or multiplayer?

Happens in both single player and multiplayer

-

No response

Version

v1.2.8.0 (Winter Update hotfix 2)

-

No response

Which operating system did you encounter this bug on?

Windows

Relevant error messages and crash reports

No response

itchyOwl commented 4 months ago

Assigned this one to you @3e849f2e5c . Please ask for further information and/or move to QA investigates, if you can't proceed with the current information.

I think the performance issues only have to be addressed if they are severe or if they are trivial to fix. I guess the notion about power might be design-related question. The last point about corrupt components seems like a bug to me.

Regalis11 commented 4 months ago

I guess the notion about power might be design-related question

In my opinion it's also a feature request, not a bug report. While I agree that it would be useful to pass power through circuit boxes, I think it would complicate the power logic so much that I feel it'd be better to keep things simple and let the circuit boxes be something you only use for signals.

Tsunder commented 3 months ago

loading this sub in my editor made baro freeze for a minute

mygamingaccount commented 2 months ago

The Baro editor warns that the number of items on the sub is way more than usual sub