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

Change clothing shop cart generation to be inside the store #51178

Merged
merged 3 commits into from
Aug 29, 2021
Merged

Change clothing shop cart generation to be inside the store #51178

merged 3 commits into from
Aug 29, 2021

Conversation

RandDenner
Copy link
Contributor

@RandDenner RandDenner commented Aug 28, 2021

Summary

Bugfixes "Fix for mis-generated cart locations in clothing shop map"

Purpose of change

Fixes #51175
Changes target terrain with mis-generated cart locations to have the carts generate in the correct places. Makes carts check for in-store generation instead of on the grass outside.

Describe the solution

Change the ASCII character potential cart generation is assigned to from a "." to " "

Describe alternatives you've considered

Changing to defined zone method other maps used. Doesn't give significant advantages (discussed in issue #51175).

Testing

Changed it locally and spawned a new shop with carts at 100%. Every potential spawn location correctly filled:
image

Makes carts check for in-store generation instead of on the grass outside.
Made minor adjustments to locations of plants and mannequins (and a wall) to prevent carts from generating in weird obstructed corners.

I may just revert all this and switch to defined zone generation instead. Thinking...
@RandDenner
Copy link
Contributor Author

I don't like those carts generating in the northwest and especially the northeast corners.
Replacing those locations with plants is a quick fix.

@actual-nh actual-nh added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display Vehicles Vehicles, parts, mechanics & interactions labels Aug 28, 2021
@kevingranade kevingranade merged commit 61aaee6 into CleverRaven:master Aug 29, 2021
@RandDenner RandDenner deleted the clothing-shop-carts branch August 29, 2021 21:31
Venera3 pushed a commit to Venera3/Cataclysm-DDA that referenced this pull request Sep 21, 2021
…ven#51178)

Makes carts check for in-store generation instead of on the grass outside.

Made minor adjustments to locations of plants and mannequins (and a wall) to prevent carts from generating in weird obstructed corners.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display Vehicles Vehicles, parts, mechanics & interactions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clothing store will only generate shopping cart outside store due to a mistake
3 participants