search

GregTechCEu / gregicality-multiblocks

(1.12) Adding many new multiblocks to GregTech CE Unofficial
MIT License
14 stars 26 forks source link

AlloyBlastSmelter can't be parallelized #27

Closed Sanduhr32 closed 1 year ago

Sanduhr32 commented 1 year ago

AlloyBlastSmelter should probably extend GCYMRecipeMapMultiblockController instead of RecipeMapMultiblockController like the LargeArcFurnace

https://github.com/GregTechCEu/gregicality-multiblocks/blob/0d8177a1ed23af65b7c79f5ba3ba8a9455f2109d/src/main/java/gregicality/multiblocks/common/metatileentities/multiblock/standard/MetaTileEntityLargeArcFurnace.java#L22

https://github.com/GregTechCEu/gregicality-multiblocks/blob/0d8177a1ed23af65b7c79f5ba3ba8a9455f2109d/src/main/java/gregicality/multiblocks/common/metatileentities/multiblock/standard/MetaTileEntityAlloyBlastSmelter.java#L49

TechLord22 commented 1 year ago

The Alloy Blast Smelter is intentionally non-parallelizable. It is meant to be an alternative to the regular Blast Furnace for alloys. It has a significant speed boost over it for base recipe durations, in addition to the added bonus of skipping the mixer step. It also has coil overclocking, which allows for huge bulk production as you upgrade the multiblock. It shouldn't need parallelization, given how fast it is already.

Additionally, combining coil overclocking with parallel recipes is incredibly overpowered, and this behavior is reserved for the Rotary Hearth Furnace.

For the above reasons, parallelization in the Alloy Blast Smelter does not make sense for GCYM, and is also not a bug. Therefore, I'm closing this issue.

Sanduhr32 commented 1 year ago

i never marked this as a bug ;( i understand that you don't want to outclass the rotary hearth furnace,

however mixing isn't that slow and you have to solidify/cool the molten alloys making that point irrelevant. thanks for explaining. A wiki or something where such intentional decisions are documented would be great.

Sanduhr32 commented 1 year ago

I've thought about this more and came to a conclusion, what about a config for parallelizing the ABF, which is false by default? Reasoning behind this is Nomifactory only allowing to craft for example Watertight Steel only through the ABF via its molten variant. (hot ingot requires dust, which requires ingot, which requires hot ingot or molten)

ghzdude commented 1 year ago

The main purpose of the Alloy Blast Smelter is to craft the alloys for GCYM. You can use the Rotary Hearth Furnace instead (along with the Mega Blast Chiller), which does accept parallelization hatches, and can do regular EBF Recipes.