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

Vested Survivor Jumpsuits And Other Minor Changes #79281

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Kantonine
Copy link
Contributor

Summary

Balance "Gives ballistic vests to survivor jumpsuits, changes names slightly, and one warmth change"

Purpose of change

Part of #74188, to try and bring the survivor items a bit more down to earth, as well as ensuring parity with other items.

Describe the solution

Adds 3.3mm of ballistic kevlar to the midweight survivor jumpsuit, heavy jumpsuit, and the firesuit. Also adds mild chainmail to the heavy survivor jumpsuit, renames the suits to survivor jumpsuits instead of kevlar jumpsuits, and makes the survivor suit slightly less warm

Describe alternatives you've considered

I did debate on whether i should leave the steel plating for the heavy jumpsuit as sheet steel, and not chainmailifying it. If there's a good reason to revert it back, i'll consider it.

Testing

Additional context

Doesn't touch any of the crafting recipes, as I lose braincells trying to change the crafting recipes without breaking or otherwise making the recipes unbalanced.

Added vests to the survivor jumpsuit, heavy survivor jumpsuit, and the firesuit. Also changed the quench tempered steel to mild chainmail for the heavy survivor jumpsuit
made the survivor jumpsuit slightly less warm
renamed the winter suits to be consistent with other changes, avoided touching the wetsuits due to planned obsoletion
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Items: Armor / Clothing Armor and clothing Game: Balance Balancing of (existing) in-game features. astyled astyled PR, label is assigned by github actions labels Jan 21, 2025
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Jan 21, 2025
@Daved27hundred
Copy link
Contributor

Daved27hundred commented Jan 21, 2025

maybe "mercenary jumpsuit" instead, so it's inline with the other Mercenary armor we have? In my eyes it rolls off the tongue better than survivor jumpsuit.

@Kantonine
Copy link
Contributor Author

maybe "mercenary jumpsuit" instead, so it's inline with the other Mercenary armor we have? In my eyes it rolls off the tongue better than survivor jumpsuit.

Hmmmm, that could work, tho it might be a bit odd with the already pre-existing mercenary coat, and the survivor trenchcoat and duster.

@IdleSol
Copy link

IdleSol commented Jan 21, 2025

  1. You have an error for the fire suit:
        "covers": [ "torso" ],
        "coverage": 100,
        "encumbrance": [ 24, 24 ]
      },
      {
        "material": [
          { "type": "nomex", "covered_by_mat": 100, "thickness": 2 },
          { "type": "kevlar", "covered_by_mat": 100, "thickness": 1.3 },
          { "type": "nylon", "covered_by_mat": 100, "thickness": 1.0 }
        ],
        "covers": [ "torso", "leg_l", "leg_r", "arm_l", "arm_r" ],

Torso in two places.

  1. I would add a "cover_vitals" parameter for suits. (Reduces damage from critical hits)

@GuardianDll
Copy link
Member

warnings for type hsurvivor_jumpsuit:
for material kevlar_layered, 3.300000 isn't a valid multiple of the thickness the underlying material comes in: 4.400000.```

@Kantonine
Copy link
Contributor Author

  1. You have an error for the fire suit:
        "covers": [ "torso" ],
        "coverage": 100,
        "encumbrance": [ 24, 24 ]
      },
      {
        "material": [
          { "type": "nomex", "covered_by_mat": 100, "thickness": 2 },
          { "type": "kevlar", "covered_by_mat": 100, "thickness": 1.3 },
          { "type": "nylon", "covered_by_mat": 100, "thickness": 1.0 }
        ],
        "covers": [ "torso", "leg_l", "leg_r", "arm_l", "arm_r" ],

Torso in two places.

2. I would add a "cover_vitals" parameter for suits. (Reduces damage from critical hits)

Thanks for the heads up! I'll add that in, how does it work, is it a percentage reduction in damage from crits, or crit reduction chance for attacks made against you?

@Kantonine
Copy link
Contributor Author

warnings for type hsurvivor_jumpsuit:
for material kevlar_layered, 3.300000 isn't a valid multiple of the thickness the underlying material comes in: 4.400000.```

I noticed that, but i don't want to end up making it thicker than it needs to be, i tried to make the ballistic kevlar layer the same as the kevlar vest, which is 3.3mm, is it a issue with the defined thickness of the suit itself, density, or what?

@GuardianDll
Copy link
Member

Need to use "ignore_sheet_thickness": true in this case

@Kantonine
Copy link
Contributor Author

Need to use "ignore_sheet_thickness": true in this case

Where does that bit go? Thnx in advance

@Maddremor
Copy link
Contributor

Why are you naming the jumpsuits to have the "survivor" prefix?

@esotericist
Copy link
Contributor

drop the renaming changes that include putting 'survivor' there. it's pointless, has an unnecessary impact on descriptiveness, and generally a bunch of us have been happy seeing its presence eroded.

Copy link
Member

@kevingranade kevingranade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has the same problem as
#78039
I'm not aware of any existing examples of plate carriers being incorporated into larger articles of clothing like this. Other articles of clothing get adjusted to go around plate carriers.

@IdleSol
Copy link

IdleSol commented Jan 22, 2025

It's most likely a translation problem. What plates are we talking about? All I see is chain mail. (lc_steel_chain)

how does it work

"cover_vitals": 10, // What percentage of critical hit damage is mitigated

I have no idea.

// Converts the "cover_vitals" protection on the specified body part into
// a modifier (between 0 and 1) that would be applied to incoming critical damage

@Kantonine
Copy link
Contributor Author

Alright, I'll revert the prefix change (and it might be carried over to when the other items get changed, or it all gets changed to mercenary if it's suitable.).

This has the same problem as #78039 I'm not aware of any existing examples of plate carriers being incorporated into larger articles of clothing like this. Other articles of clothing get adjusted to go around plate carriers.

I'm not having plates for the suits, that stays exclusive to the plate carriers (and mercenary coat) the chainmail is for additional protection for the arms and legs. I am not using the plate carriers, but rather the plate-less kevlar vest (https://cdda-guide.nornagon.net/item/kevlar), the same one used for the mercenary body armor. I can see that there might be more lee-way with the mercenary body armor, seeing that it's only a combat shirt, kevlar vest, and kevlar arm protection all stitched together, making it quite a bit thinner overall compared to the jumpsuits.

@tenehea
Copy link

tenehea commented Jan 23, 2025

If the player wanted to wear chainmail with their kevlar jumpsuit, they'd make a kevlar jumpsuit and a suit of chainmail..

@GuardianDll
Copy link
Member

Where does that bit go?

like this

{ "type": "kevlar_layered", "thickness": 1.1, "ignore_sheet_thickness": true }

@kevingranade
Copy link
Member

Plate, chain, whatever, who is attaching metal armor sections to otherwise regular clothes? Clothes are clothes and armor is armor.

@tenehea
Copy link

tenehea commented Jan 23, 2025

Plate, chain, whatever, who is attaching metal armor sections to otherwise regular clothes? Clothes are clothes and armor is armor.

?

All armor is clothing but not all clothing is armor. A coat of plates, for example, is clothing (a cloth tunic) with plates riveted on top.

@Kantonine
Copy link
Contributor Author

Plate, chain, whatever, who is attaching metal armor sections to otherwise regular clothes? Clothes are clothes and armor is armor.

Our CDDA survivors/NPCs sure do:
https://cdda-guide.nornagon.net/item/jeans_mod
https://cdda-guide.nornagon.net/item/vest_jean_mod
https://cdda-guide.nornagon.net/item/jacket_jean_mod
https://cdda-guide.nornagon.net/item/vest_leather_mod
https://cdda-guide.nornagon.net/item/jacket_leather_mod
https://cdda-guide.nornagon.net/item/gloves_fingerless_mod

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 Game: Balance Balancing of (existing) in-game features. Items: Armor / Clothing Armor and clothing [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants