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

First Version Of Encumbrance By Sentence #57243

Merged

Conversation

bombasticSlacks
Copy link
Contributor

@bombasticSlacks bombasticSlacks commented Apr 29, 2022

Summary

None

Purpose of change

@Aerin-of-the-Toast mentioned that they wanted to overhaul all the helmets in the game since they found research that the major thing that effects helmet encumbrance is weight. They created a chart with some brackets based on weight and them some flat modifiers so that they could fairly modify all helmets... This sounds like part of what I want to do in the future to all encumbrance values so I figured I could build some preliminary infrastructure now so helmets should be overhauled in the desired uniform way while leading to further work in the future.

Describe the solution

Limbs can now describe a set of fixed weight to encumbrance values for the limb in json. These can then be used as reference points to interpolate other weight to encumbrance values.

When defining an armor entry instead of giving a number for encumbrance you can give a set of encumberance_modifiers.

This will make it so encumbrance is calculated by doing a linear interpolation on the two closest weight to encumbrance values for the limb and then modifying it by any encumbrance_modifiers that have been provided for the item.

The encumbrance modifiers for helmets are:

  • IMBALANCED +10
  • RESTRICTS_NECK +10
  • WELL_SUPPORTED -10
  • NONE +0

Otherwise everything just works ™️ still.

Describe alternatives you've considered

Testing

image
it works for the army helmet and motorcycle helmet I added in this PR.

Additional context

I'm starting this as draft for now so I can get some feedback and write some documentation tomorrow.

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON Items: Armor / Clothing Armor and clothing Limbs Limbs, mutable limbs, and code related to them. astyled astyled PR, label is assigned by github actions labels Apr 29, 2022
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Apr 29, 2022
@bombasticSlacks
Copy link
Contributor Author

Note for me for tomorrow make sure a negative number isn't possible.

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Apr 29, 2022
@github-actions github-actions bot removed the BasicBuildPassed This PR builds correctly, label assigned by github actions label Apr 30, 2022
@github-actions github-actions bot added <Documentation> Design documents, internal info, guides and help. [Markdown] Markdown issues and PRs labels Apr 30, 2022
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Apr 30, 2022
@bombasticSlacks bombasticSlacks marked this pull request as ready for review April 30, 2022 18:16
@kevingranade kevingranade merged commit a26d6be into CleverRaven:master Apr 30, 2022
@omni-Var
Copy link

omni-Var commented May 1, 2022

Are these values offset by strength scores, since it's all about weight now? The football player obviously has an easier time fielding this encumbering weight than the office nerd that never works out.

@bombasticSlacks
Copy link
Contributor Author

the first implementation will just be for helmets, which doesn't have much to do with strength. In the future when the system is expanded, I'm not sure.

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` <Documentation> Design documents, internal info, guides and help. Items: Armor / Clothing Armor and clothing [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Limbs Limbs, mutable limbs, and code related to them. [Markdown] Markdown issues and PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants