search

stfwi / engineers-decor

Engineer's Decor
MIT License
35 stars 16 forks source link

Small Mineral Melting Furnace loosing Lava output #92

Closed papaworld closed 4 years ago

papaworld commented 4 years ago

MC: 1.15.2 FML: 31.1.30 Engineer's Decor: engineersdecor-1.15.2-1.0.20-b2

Using Mekanism Basic or Advanced Mechanical Pipe to remove Lava from Melting Furnace... I am only getting 100mb of the 1,000mb produced. I brought this issue up with the Mekanism team and they responded with the following:

The fluid voiding is a bug in Engineer's decor. (I am leaving this issue open for now to track/fix the fallback for before configs load that I found while looking for the issue.) From what I can tell: https://github.com/stfwi/engineers-decor/blob/master/1.15/src/main/java/wile/engineersdecor/blocks/BlockDecorMineralSmelter.java#L494 they are ignoring the amount we request to drain and just are draining whatever is stored from their mineral smelter. The reason it worked before, is that their other drain method seems to be implemented properly, and we used to use that before I switched to the fluid stack specific one to have proper support for tile's that have more than one tank.

Is it possible to fix the issue? My Lava based smelting, FE power generators and bucket fillers no longer work.

Thank you

stfwi commented 4 years ago

Hi, aye, that is quite a glitch ;-). I'll schedule that for the weekend. Cheers -

stfwi commented 4 years ago

(Related issue https://github.com/mekanism/Mekanism/issues/6011).

stfwi commented 4 years ago

Like that ? ;-)

2020-04-01_19 17 15

papaworld commented 4 years ago

Hello, yes for the most part. Melter Lava output bottom to 1 piece of Mekanism Basic Mechanical Pipe (100mb pull rate). Used the configurator to set pipe to pull from Melter. Then to a tank. The tank receives only 100mb per Melt operation. (I did not use the wheel check valve.) With Mekanism Advanced Mech Pipe (400mb pull rate) the tank receives 400mb per operation.

BTW: really helpful mod, nice textures too.

papaworld commented 4 years ago

Found out if the Mechanical Pipe was not in pull mode... left at default connection, the amount transferred is correct. Thank you for your help. Enjoy!

stfwi commented 4 years ago

Cheers, the fix release is scheduled for this weekend (just need to double check with my local server again).

stfwi commented 4 years ago

Alright, the file is approved on curse. After my tests were ok, could you also make a quick check if the problem is resolved for your setup and close the issue if so? Cheers -