[boskid][0.18.34] FBCompound renders pipes and covers using GFX from first FB

[Adamo]

Hey All,

I found an (I suspect) esoteric bug. In the case where a production machine has two input fluid boxes -- one being of underground type designated by setting the max_underground_distance value, and one being of the normal above-ground type -- AND both inputs are accepting a single, shared ingredient, the pipe pictures assigned to the above ground pipe do not render. In the case where two ingredients are being accepted (so each of the two inputs I mentioned are accepting different ingredients), the pipe pictures do render. Also in the case where I limit the input to a single input using a fluidbox_index on the ingredient, the pipe pictures do render.

Thanks!

[posila]

Hello, thanks for the report.
Could you please provide a save and a mod (if it is not on available mod portal) that we can load to see the issue?

[Adamo]

Sure thing. Take a peek at this save, which is just a simple proving ground. I *think* you can see the issue with just the following mods: adamo-carbon and adamo-cinefaction. I'll upload those here. Hopefully with those loaded you can see the issue right when you load up. Peek at the "Still" entity after setting it to the Basic Gas Processing recipe. The inputs/outputs are a little confusing because the underground pipe overlaps an aboveground pipe. The 2 inputs are the one above ground input and the underground input, and they are on opposite sides of the entity. Hook up a pipe to the input and the outputs, and you should be able to see that the pipe cap is not rendering. Change the recipe to something that uses 2 inputs, like Advanced Processing, and you should see the pipe cap be rendered again.

[boskid]

From what i see, issue is quite convoluted. That entity has 2 fluid boxes for input, first one is with 2 underground connections and no pipe covers (same applies to pipe pictures), second has 1 regular connection and has pipe covers. When multiple fluidboxes are merged due to recipe, they form a FluidBoxCompound. This compound uses pipe covers from first covered fluidbox and since you have no pipe covers for it, it does not draw any covers for that fluidbox compound. Possible workarounds are to add pipe covers to that fluidbox even if it does not use them.

[boskid]

Thanks for the report. It is now fixed for 0.18.35. Pipe connections will use proper pipe pictures and covers from fluid box prototype they were defined in.

[Adamo]

From what i see, issue is quite convoluted. That entity has 2 fluid boxes for input, first one is with 2 underground connections and no pipe covers (same applies to pipe pictures), second has 1 regular connection and has pipe covers. When multiple fluidboxes are merged due to recipe, they form a FluidBoxCompound. This compound uses pipe covers from first covered fluidbox and since you have no pipe covers for it, it does not draw any covers for that fluidbox compound. Possible workarounds are to add pipe covers to that fluidbox even if it does not use them.

Oh, that makes sense. Didn't think to put pipe covers on those pipes, but I can see now how that might have worked. Yeah, it's convoluted. Sounds like you found a fix regardless, really appreciate it!