Skip to content
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

New NPCs for the player to trade with #37347

Closed
wants to merge 3 commits into from

Conversation

Brytenwalda
Copy link
Contributor

Summary

SUMMARY: Content "New NPCs for the player to trade with"

Purpose of change

I added Free Merchant traders to gas stations and hunters to cabins and campsites to add some variety to the game - players will now have NPCs to trade and interact with. They should be rather lore-friendly and expand the lore re Free Merchants.

Describe the solution

I modified mapgen files for cabins, gas stations and campsites to add new NPCs and modified npc files to put new types in game.

Testing

I encountered no issues in-game and I've been using these NPCs for a while now.

@curstwist
Copy link
Contributor

I'd recommend moving the place_npcs to a nested mapgen, then placing the nested map in the locations. This way, you can set the NPC to a spawn chance instead of being there 100% of the time.

You can see examples of NPC nested mapgen at: https://github.com/CleverRaven/Cataclysm-DDA/blob/master/data/json/mapgen/nested/city_npc_nested_spawns.json

You can simply put the place_npcs entry into the nest's object entry and nothing else, or, take the opportunity to also change what is spawning around it in the locations (say more storage, their base area like a bed, brazier, etc).

the main mapgen entry gets something like this:
"place_nested": [ { "chunks": [ [ "chef_s_restaurant", 5 ], [ "null", 95 ] ], "x": 7, "y": 18 } ]

This has my chef spawning 5% of the time and 95% they restaurant will be empty.

Thanks a lot for adding more NPCs!

@curstwist curstwist added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display NPC / Factions NPCs, AI, Speech, Factions, Ownership labels Jan 24, 2020
@Night-Pryanik
Copy link
Contributor

I'd recommend moving the place_npcs to a nested mapgen, then placing the nested map in the locations. This way, you can set the NPC to a spawn chance instead of being there 100% of the time.

We really need an ability to directly set chance to spawn for NPCs.

@ghost
Copy link

ghost commented Apr 10, 2020

Moved to #39447.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants