dont simulate fluid filling placements in break sim. The place sim already does that if its required

Author: beanbag44

src/main/kotlin/com/lambda/config/AutomationConfig.kt

@@ -79,7 +79,6 @@ open class AutomationConfig(
             val showAllEntries by setting("Show All Entries", false, "Show all entries in the task tree").group(Group.Debug)
             val shrinkFactor by setting("Shrink Factor", 0.001, 0.0..1.0, 0.001).group(Group.Debug)
             val ignoreItemDropWarnings by setting("Ignore Drop Warnings", false, "Hides the item drop warnings from the break manager").group(Group.Debug)
-            val maxSimDependencies by setting("Max Sim Dependencies", 3, 0..10, 1, "Maximum dependency build results").group(Group.Debug)
 
             @Volatile
             var drawables = listOf<Drawable>()

src/main/kotlin/com/lambda/config/groups/BuildConfig.kt

@@ -29,6 +29,7 @@ interface BuildConfig {
     val interactionsPerTick: Int
     val maxPendingInteractions: Int
     val interactionTimeout: Int
+    val maxBuildDependencies: Int
 
     val useDefaultReach: Boolean
     val attackReach: Double

src/main/kotlin/com/lambda/config/groups/BuildSettings.kt

@@ -40,6 +40,7 @@ class BuildSettings(
     override val interactionsPerTick by c.setting("Interactions Per Tick", 5, 1..30, 1, "The amount of interactions that can happen per tick").group(baseGroup, Group.General).index()
     override val maxPendingInteractions by c.setting("Max Pending Interactions", 15, 1..30, 1, "The maximum count of pending interactions to allow before pausing future interactions").group(baseGroup, Group.General).index()
     override val interactionTimeout by c.setting("Interaction Timeout", 10, 1..30, 1, "Timeout for block breaks in ticks", unit = " ticks").group(baseGroup, Group.General).index()
+    override val maxBuildDependencies by c.setting("Max Sim Dependencies", 3, 0..10, 1, "Maximum dependency build results").group(baseGroup, Group.General).index()
 
     override val useDefaultReach by c.setting("Default Reach", true, "Whether to use vanilla interaction ranges").group(baseGroup, Group.Reach).index()
     override val attackReach by c.setting("Attack Reach", DEFAULT_ATTACK_REACH, 1.0..10.0, 0.01, "Maximum entity interaction distance") { !useDefaultReach }.group(baseGroup, Group.Reach).index()

src/main/kotlin/com/lambda/interaction/construction/simulation/Sim.kt

@@ -17,7 +17,6 @@
 
 package com.lambda.interaction.construction.simulation
 
-import com.lambda.config.AutomationConfig
 import com.lambda.interaction.construction.processing.PreProcessingInfo
 import com.lambda.interaction.construction.result.BuildResult
 import com.lambda.interaction.construction.result.results.GenericResult
@@ -77,7 +76,7 @@ abstract class Sim<T : BuildResult> : Results<T> {
      */
     protected suspend fun ISimInfo.withDependent(dependent: Sim<*>, block: suspend () -> Unit) {
         // +1 because the build sim counts as a dependent
-        if (dependencyStack.size >= AutomationConfig.Companion.DEFAULT.maxSimDependencies + 1) return
+        if (dependencyStack.size >= buildConfig.maxBuildDependencies + 1) return
         dependencyStack.push(dependent)
         block()
         dependencyStack.pop()

src/main/kotlin/com/lambda/interaction/construction/simulation/checks/BreakSim.kt

@@ -277,7 +277,6 @@ class BreakSim private constructor(simInfo: ISimInfo)
 
             affectedFluids.forEach { (liquidPos, liquidState) ->
                 result(BreakResult.Submerge(liquidPos, liquidState))
-                sim(liquidPos, liquidState, TargetState.Solid(emptySet())) { simPlacement() }
             }
             result(BreakResult.BlockedByFluid(pos, state, affectedFluids.keys))
             return true