View Issue Details

IDProjectCategoryView StatusLast Update
0000258Industrial-Craft²crossmod-compatibility / submodules / APIpublic2013-09-22 10:24
Reporterretep998 Assigned ToThunderdark  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionno change required 
Summary0000258: Geothermals do not connect to RedPower fluid Pipes
DescriptionGeothermal Generators are able to accept lava using Buildcraft waterproof pipes.
However, they are not able to connect to RedPower2 fluid pipes.
In contrast, the thermal generators from GregTech are able to connect to both Buildcraft and RedPower2 pipes.
Attached is a screenshot comparing the thermal and generators with redpower2 and buildcraft pipes.
Testing was performed both on ssp and smp using MC 1.4.6 with IndustrialCraft 1.112.192, BuildCraft 3.3.0 and RedPower 2.0pr6.
Steps To Reproduce1. Place a Geothermal Generator
2. Place a redpower fluid pipe next to it
3. Observe how they do not connect at all
TagsNo tags attached.
Minecraft Version

Activities

retep998

2013-01-13 16:32

reporter  

2013-01-13_11.23.43.png (367,383 bytes)

Wliu

2013-01-15 22:16

reporter   ~0000652

Try BC 3.4.0.

retep998

2013-01-16 02:48

reporter   ~0000653

Last edited: 2013-01-16 02:48

The problem here is not with BuildCraft compatibility, but with RedPower compatibility.
Other mods such as Railcraft and GregTech use the Forge Liquid API and are fully compatible with RedPower's fluid pipes, while BuildCraft and IndustrialCraft use Buildcraft's liquid API system and are thus incompatible with RedPower.
If GregTech can get it right, and it's an IC2 addon, then clearly IndustrialCraft itself should be able to get it right as well.
Anyway, I upgraded to BC 3.4.0 but it didn't make any difference.

narc

2013-01-19 01:17

reporter   ~0000668

Last edited: 2013-01-19 01:18

There is no Buildcraft liquid API, and there hasn't been one since Forge incorporated it.

Redpower is unusual in how it decides whether to connect its fluid pipes -- it uses ITankContainer.getTank(direction, null) rather than .getTanks(direction) -- presumably because Eloraam didn't feel a need to pull an array of tanks just to check if one exists, or something.

The easiest fix from the IC2 side would be to allow a null LiquidStack on getTank, which is explicitly mentioned as permitted by the Forge javadocs anyway.

Thunderdark

2013-09-22 10:24

viewer   ~0001519

clean up Manits Bug Tracker..close automatic all Bug report > 6 months if the error still exists in the current version > IC2 2.0.217 please report new Thx.

Issue History

Date Modified Username Field Change
2013-01-13 16:32 retep998 New Issue
2013-01-13 16:32 retep998 File Added: 2013-01-13_11.23.43.png
2013-01-15 22:16 Wliu Note Added: 0000652
2013-01-16 02:48 retep998 Note Added: 0000653
2013-01-16 02:48 retep998 Note Edited: 0000653
2013-01-19 01:17 narc Note Added: 0000668
2013-01-19 01:18 narc Note Edited: 0000668
2013-09-22 10:24 Thunderdark Note Added: 0001519
2013-09-22 10:24 Thunderdark Status new => closed
2013-09-22 10:24 Thunderdark Assigned To => Thunderdark
2013-09-22 10:24 Thunderdark Resolution open => no change required