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

Restore the damage values of the machete, combat machete, and qt combat machete after some unintended nerfs #76168

Closed

Conversation

Regilith
Copy link
Contributor

@Regilith Regilith commented Sep 3, 2024

Summary

Balance "Restore the damage values of the machete, combat machete, and qt combat machete after some unintended nerfs"

Purpose of change

Fixes #76152

The machete, combat machete, and qt combat machete have all had their damages significantly reduced by some PRs, and these changes appear to have been unintended. The current DPS of the qt variant in particular is worse than the two bronze equivalent shortswords - the xiphos and the khopesh - despite them all being similar weights and lengths, and of the latter two being drastically worse material quality (bronze versus tempered steel), and significantly worse than all the other qt options in shortswords_dps.json

data/mods/TEST_DATA/expected_dps_data/shortswords_dps.json

"machete": 16.9
"survivor_machete": 17.41
"survivor_machete_qt": 20.1
"sword_xiphos": 22.0
"khopesh": 24.26
"qt_cutlass": 26.3
"qt_scimitar": 27.0
"qt_wakizashi": 27.68
"qt_cavalry_sabre": 28.7
"qt_rapier": 30.5

The two reasons I found for this were in these PRs:

#66956 changed the absolute damage values of the combat machetes to use relative instead, but instead of preserving the 28 cut damage of the qt variant, it unintendedly reduced it to 26 cut, and one less bash due to the flag not taking into the account the relative damage of the regular combat machete, which the qt copy-from's. You can see this current cut damage in game (see #76152 for images, the item browser is wrong).

#68146 removed token bash damage from knives, but the machete, combat machete, and qt combat machete were also affected by this change despite them being short swords, not knives. This was probably due to the machete having "knife" in its description. Either this was intended, and the other short swords should have their bash damage removed since they have similar lengths and weights to the machetes (the xiphos is 10cm shorter, for example), or (more likely) this was accidental, as it left machetes as the only SHORT_SWORDS without bash damage.

Describe the solution

Restore the damage values to what they were before the PRs, to be more in-line with the other short swords in the DPS list. The qt variant now has the same damage as a xiphos, but is faster to swing.

Testing

Just numbers changes.

image

Additional Context

After this PR, the new expected DPS is this (see 1cd5bc4 for where they place compared to other short swords):

"machete": 19.34,
"survivor_machete": 20.64,
"survivor_machete_qt": 25.8,

For some reason, "replace_materials": { "steel": "qt_steel", "wood": "kevlar" }, isn't working correctly either for the qt machete, it shows as having wood instead of kevlar in-game. It seems some of these json inheritance flags don't correctly work when you chain copy-from's together.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON <Bugfix> This is a fix for a bug (or closes open issue) Game: Balance Balancing of (existing) in-game features. astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Sep 3, 2024
@Regilith Regilith changed the title Restore the damages of the machete, combat machete, and qt combat machete after some unintended nerfs Restore the damage values of the machete, combat machete, and qt combat machete after some unintended nerfs Sep 3, 2024
@github-actions github-actions bot added the Code: Tests Measurement, self-control, statistics, balancing. label Sep 3, 2024
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.

I'd say this is the thing to do: "the other short swords should have their bash damage removed".
Overall regardless of how it happened, I think it's totally fine for machetes to fall short of dedicated swords in combat capability. They're not made for fighting, they're made for chopping grass and shrubs and being cheap.

@Regilith
Copy link
Contributor Author

Regilith commented Sep 3, 2024

They're not made for fighting, they're made for chopping grass and shrubs and being cheap.

This is true for generic irl machetes, and our generic machete should probably have reduced damage / less durability, but at least for our machete in-game it is described as being based off of the OKC 1-18, which is described as a "Military Machete."

The combat machete / qt combat machete are specifically described as being:

"customized and rebalanced to improve its performance as a weapon"
"customized, rebalanced and tempered to dramatically improve its performance as a weapon."

So these aren't generic machetes anymore, they're closer to 'zombie choppers' that some irl cutlery companies produce. If we want to actually have an end-game weapon in the SHORT_SWORDS category, we might want to reflavor the combat machetes to be totally different things, rather than modified gardening machetes.

I think it's totally fine for machetes to fall short of dedicated swords in combat capability.

Even with the adjusted damages, the qt combat machete still falls short of all the other qt swords in that DPS list, and the combat machete is around the same damage as a bayonet or sword cane, which seems reasonable.

My suggestions:

  1. Change the current machete to be based on more of a more generic, grass-cutting, store-bought variety. Reduce its damage, remove the STURDY flag, change its length/weight to whatever generic reference we pick, update the description.

  2. Change the acquisition of the combat machete to be either hand-forged from scratch, or as a rare drop in some locations (feral preppers and prepper houses come to mind), since the current recipe of super glue and grinding doesn't really make sense to make a machete more combat-capable. Base this machete off of the okc 1-18 or some equivalent 'zombie chopper'. Update the description to reflect how much different it is from a normal machete.

  3. Keep the tempered steel combat machete as a high-fab crafting exclusive. Replace the description to distance it from the garden variety of machete.

  4. Reduce/remove bash since these are lighter than most swords, possibly compensate with higher cut.

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Sep 3, 2024
@Regilith
Copy link
Contributor Author

Regilith commented Sep 4, 2024

I'll rethink this and possibly re-approach it.

@Regilith Regilith closed this Sep 4, 2024
@kevingranade
Copy link
Member

based off of the OKC 1-18, which is described as a "Military Machete."

Just because it's military doesn't mean it's a good weapon???

"customized and rebalanced to improve its performance as a weapon"

You're only getting so far by tempering bad steel and adjusting the profile of a machete, it's still a machete. It's a thin profiled broad blade made of a metal alloy that is optimized for not weapon usage.

they're closer to 'zombie choppers' that some irl cutlery companies produce.

Which are themselves cosplaying as weapons.

If we want to actually have an end-game weapon in the SHORT_SWORDS category

We don't, a short sword is fundamentally not an end game weapon, and machetes sit below the real short swords.

to be either hand-forged from scratch, or as a rare drop in some locations

You're missing the point here, if you extensively resesign a machete to be a capable short sword, it's no longer a machete. If it's still a machete, it's not a sword. If it's a sword... it's a sword, which we already have.

Please just drop it, machetes are a dead end. If you want a sword just use a sword.

@Regilith Regilith deleted the combat-machete-damage-restore branch September 5, 2024 17:37
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 <Bugfix> This is a fix for a bug (or closes open issue) Code: Tests Measurement, self-control, statistics, balancing. Game: Balance Balancing of (existing) in-game features. [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.

relative json inheritance might be wonky for survivor_machete_qt
2 participants