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

Update s_clothes chances #36823

Merged
merged 1 commit into from
Jan 8, 2020
Merged

Conversation

ymber
Copy link
Member

@ymber ymber commented Jan 8, 2020

Summary

SUMMARY: Content "Update clothing store distributions"

Purpose of change

All s_clothes variations had the same chance of appearing. Some of them should be much less common than others.

Describe the solution

Fix distribution of s_clothes variations.

Testing

Spawn rates of different variations are as expected.

@Night-Pryanik
Copy link
Contributor

All s_clothes variations had the same chance of appearing. Some of them should be much less common than others.

Please do tell us in details why exactly you think this store should spawn more frequently and that store should spawn less frequently.

@ymber
Copy link
Member Author

ymber commented Jan 8, 2020

The most common ones now are the average high street clothes stores. The expensive tailoring and fur places are now less common and I weighted the tailoring spawn chances towards the recently updated variations because the old ones need item spawns fixing. Generally the changes reflect the average western market for clothes stores.

@I-am-Erk I-am-Erk added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display labels Jan 8, 2020
@I-am-Erk I-am-Erk merged commit bb7e3c4 into CleverRaven:master Jan 8, 2020
@ymber ymber deleted the clothing_store_chances branch January 8, 2020 17:35
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants