As I understand, the shallow (walkable) water terrain should not allow any buildings on top of it, however, it is currently possible to place an offshore pump on the interface between it and water (but not connect any pipes to it).
[0.17.48] Possible to put offshore pump on the edge of shallow/deep water
[0.17.48] Possible to put offshore pump on the edge of shallow/deep water
Leading Hebrew translator of Factorio.
Re: [0.17.48] Possible to put offshore pump on the edge of shallow/deep water
Thanks for the report.
As Twinsen would say ... I fucked myself. Offshore pump is built on water (not on ground), but it has check that tile adjacent to it's pipe output has ground collision layer.
I really, really want to avoid adding new collision layers, as it would mean I would have to remove one of the few unused layers exposed to mods (layer-11 to layer-15), I absolutely didn't want to do full pipe buildability check and it made sense to me at that time to do it as "collision test" - test if tile has collision layer(s), as opposed to using regular "collision mask" which would test the tile doesn't collide with given layers. So solution would be to change the fluid_box_tile_collision_test to fluid_box_tile_collision_mask, but I don't want to bring more wrath of modders onto us, just because of this minor issue, so I'll do it after 0.17 is stable for future releases.
As Twinsen would say ... I fucked myself. Offshore pump is built on water (not on ground), but it has check that tile adjacent to it's pipe output has ground collision layer.
I really, really want to avoid adding new collision layers, as it would mean I would have to remove one of the few unused layers exposed to mods (layer-11 to layer-15), I absolutely didn't want to do full pipe buildability check and it made sense to me at that time to do it as "collision test" - test if tile has collision layer(s), as opposed to using regular "collision mask" which would test the tile doesn't collide with given layers. So solution would be to change the fluid_box_tile_collision_test to fluid_box_tile_collision_mask, but I don't want to bring more wrath of modders onto us, just because of this minor issue, so I'll do it after 0.17 is stable for future releases.
- eradicator
- Smart Inserter
- Posts: 5211
- Joined: Tue Jul 12, 2016 9:03 am
- Contact:
Re: [0.17.48] Possible to put offshore pump on the edge of shallow/deep water
The amount of wrath is linearly dependant on the usefulness of the "fix" to modders. If the fix happens to solve the sandpit only on sand problem you could easily generate negative wrath . (I couldn't resist to make that joke...)
Author of: Belt Planner, Hand Crank Generator, Screenshot Maker, /sudo and more.
Mod support languages: 日本語, Deutsch, English
My code in the post above is dedicated to the public domain under CC0.
Mod support languages: 日本語, Deutsch, English
My code in the post above is dedicated to the public domain under CC0.
Re: [0.17.48] Possible to put offshore pump on the edge of shallow/deep water
This problem appears to be fixed by the introduction of the new pumps - feel free to move to "resolved".
Leading Hebrew translator of Factorio.