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

Add trait property to supress traits being granted at random in chargen #77953

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

kevingranade
Copy link
Member

Summary

None

Purpose of change

Got a report of NPCs spawning with unexpected traits after #76935, which I believe expanded the traits that NPCs can have assigned to them.

Describe the solution

Add a property to mutations indicating they shouldn't be acquired at random at chargen time.
Adjusted random selection at chargen time to filter on this property.
Assigned new property to traits that shouldn't be randomly assigned, either to players or NPCs.

Testing

Need to run a quick test to generate a bunch of random NPCs and enumerate the traits they spawn with.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies [C++] Changes (can be) made in C++. Previously named `Code` Character / World Generation Issues and enhancements concerning stages of creating a character or a world json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Nov 17, 2024
@kevingranade kevingranade force-pushed the supress-traits-at-random-chargen branch from 2d52f69 to bf3df08 Compare November 18, 2024 02:03
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Nov 18, 2024
@Night-Pryanik Night-Pryanik merged commit 61bd664 into master Nov 18, 2024
21 of 27 checks passed
@RedMisao RedMisao mentioned this pull request Nov 28, 2024
@kevingranade kevingranade deleted the supress-traits-at-random-chargen branch December 1, 2024 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` Character / World Generation Issues and enhancements concerning stages of creating a character or a world [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants