-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove salt water from swamps #78419
base: master
Are you sure you want to change the base?
Conversation
If you remove the salt water swamps you should also remove the salt water pipe/pump from the workshop 2 expansion, as well as the salt water pump. |
@PatrikLundell I'm not really familiar with faction camp stuff, instead of deleting do you know if these could be updated with reasonable effort to work on the sea shore where salt water is still available? I took a quick look at the recipes and I'm mildly scared, and it isn't immediately obvious to me where the definition is for what OMT these are buildable on. |
This will have a huge impact on Innawood, since salt is required for curing leather (which, in turn, is required for pottery and affordable clothing). Without salt water from swamps, I believe the only source of salt would be occasional rock salt in caves. Perhaps this should be left in for Innawood specifically (while breaking realism and consistency with vanilla)? |
Yeah no problem doing that for Innawood if that's needed. |
The salt water pipe logic looks for a swamp tile and then plunks down a well in a fixed location in that tile. That would be wonky to do with a ocean shore tile, as you wouldn't really know whether it would be built on land or on the water. Finally, making it available close to the ocean only would be a rather limited use case, not the least because that's where the cities are. Ripping the pipe out might be a bit tricky, because there might be people who are in the process of building it, and there might be people who've planned to make use of a swamp eventually (and would be able to do it, because the swamp has already been generated, and thus already has salt water in it). You don't really have to do anything to the pump or the well (I think there's a special salt water version of it as well), since they're only used by this functionality, and thus have no other usages. The functionality of the salt water pipe construction is that it allows you to build a pipe between a swamp at some maximum distance to the workshop expansion version 2 by generating a number of construction tasks consuming pipes (and fittings) to simulate the construction of a pipe between the swamp and a pump at the expansion. This pipe can only cross a fixed set of terrain (forest, field, road, and possibly a few more). The process first generates a job to create a well in the swamp and a simulated pipe to the edge of it, and then jobs to create simulated pipe sections, until the simulated pipe reaches the edge of the workshop tile, after which the last job adds a simulated pipe and a pump. |
This would make Innawood runs near impossible unless you find an ocean, so I would appreciate it if you kept the salty bogs in Innawood, yes. |
Summary
Content "Remove salt water from swamps"
Purpose of change
Remove random pools of salt water from swamps. Salt water swamps don't really exist in NE, there's some coastal salt marshes but our general swamp terrain doesn't represent that.
Describe the solution
Switch t_swater terrains with t_water in vanilla, TropiCata and Rural Mapgen mods since these terrains were obviously just copypasted from vanilla. Happy to switch it back for TropiCata if requested.
Describe alternatives you've considered
Testing
Generate a world and check there's no more random pools of salt water in swamps.
Additional context