Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign upTo add Cosmic's Additional Locations to the game #24576
Conversation
cosmiccoincidence
added some commits
Jul 31, 2018
This comment has been minimized.
This comment has been minimized.
|
I'm not sold on the empty lot terrains. They take city space that could be used by interesting and usefull buildings and replace them with what is basically nothing, Our cities already generate with what are tentatively empty lots (AKA mid city empty fields)Maybe consider editing mapgen code so that those fields become empty lots? |
cosmiccoincidence
added some commits
Aug 1, 2018
cosmiccoincidence
added some commits
Aug 1, 2018
This comment has been minimized.
This comment has been minimized.
|
I don't know how to directly reply but responding to the empty lots. I definitely hear and understand that argument, I tried to offset that from being too much of an issue by keeping most of spawn/occurrence rates for those empty-ish (lots) and mostly-for-flavor (tree farm) locations low. Also the empty lots although not as interesting as most places, can spawn trash, deserted vehicles, food vendor carts, so they aren't all completely empty. Also I just linted ALL my code as well as threw in a couple other buildings... Landscaping Supply Co, LAN Center, and Dollar Store |
cosmiccoincidence
added some commits
Aug 1, 2018
This comment has been minimized.
This comment has been minimized.
|
Jenkins rebuild. |
ZhilkinSerg
reviewed
Aug 1, 2018
| } | ||
| } | ||
| } | ||
| ] |
This comment has been minimized.
This comment has been minimized.
ZhilkinSerg
Aug 1, 2018
Contributor
You have EOL missing in the end of several files, so JSON formatting fails.
ZhilkinSerg
reviewed
Aug 1, 2018
| "type": "mapgen", | ||
| "weight": 100 | ||
| } | ||
| ] |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Zireael07
Aug 1, 2018
Contributor
End of line mark, usually inserted by your text editor. What do you use?
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Nymall
Aug 1, 2018
Contributor
Just put a extra row at the end of the JSON and it should fix the issue
This comment has been minimized.
This comment has been minimized.
cosmiccoincidence
added some commits
Aug 1, 2018
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Warehouse garage doors fixed. |
ZhilkinSerg
merged commit 93347ad
into
CleverRaven:master
Aug 1, 2018
cosmiccoincidence
deleted the
cosmiccoincidence:cosmic's_additional_locations
branch
Aug 3, 2018
nexusmrsep
added this to Done
in Mapgen and overmapgen improvements
Sep 3, 2018
ZhilkinSerg
referenced this pull request
Nov 29, 2018
Closed
Linked More Buildings and Locations Mod needs "forest_mapgen_settings" #234
This comment has been minimized.
This comment has been minimized.
|
This pull request has been mentioned on Cataclysm: Dark Days Ahead. There might be relevant details there: https://discourse.cataclysmdda.org/t/potential-engine-solutions-and-fun-stuff/17904/13 |

cosmiccoincidence commentedAug 1, 2018
•
edited
This request is to add my (CosmicCoincidence) additional location mods to the base game (not as official mods but part of the vanilla game). These additional locations include...