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

fix(balance): Recipe changes to address land mine component inconsistencies #4389

Merged
merged 5 commits into from
Mar 26, 2024

Conversation

NerdPunkFu
Copy link
Contributor

@NerdPunkFu NerdPunkFu commented Mar 25, 2024

Purpose of change

fix #4381

To fix land mine component inconsistencies in regard to real life land mines and other explosives within the game. For more issue see #4381. To fully address the issue land mines still need to be divided into real world or fictional models, but this requires the land mine trap effect to be moved from an hardcoded effect to something that can be modified by JSON.

Describe the solution

Introduces my changes to the crafting and uncrafting recipes for land mines mentioned in #4381. For crafting I added bone glue, changed the fragmentation element to be the same as the scrap loaded 00 shot recipe just 10x to be in line with what it was before, got rid of all that random nonsense that were useable as explosive and instead used the same lists as other explosives used equivalent in amount to 600 composition b and lastly I added the option to use fuses for landmines since for example the M16 mine uses a black powder fuse(there should be a mechanism to light it when stepped on so maybe a "makeshift fuze" item should be added that is made of fuse and some method to light it e.g. flint and steel). For disassembly I just increased the composition b amount to 600 and added bearings as frag.

Describe alternatives you've considered

Rewrite how the game handles land mine traps, but this requires rewriting the hardcoded effect land mine traps currently have. Also, there is ambiguity about what kind of land mine the ingame landmines are supposed to be, I just assumed it's a classic explodes-in-the-ground type of land mine that has a charge about as strong as a claymore or M16 mine, but AFAIK the US military does not use such mines and none of them have composition b as main charges, but we do not have TNT or C4 in increments that would make it useable in crafting like we do for composition b or other explosives like RDX. We should probably also address this deficiency in the future, no reason why a player could take bits of C4 to make custom charges. It's basically a hard, chewy playdough style paste. TNT is harder, but probably still possible?

Testing

Got no errors booting up, crafting or uncrafting in my playthrough using these changes.

Additional context

Checklist

Copy link
Contributor

autofix-ci bot commented Mar 25, 2024

Autofix has formatted code style violation in this PR.

I edit commits locally (e.g: git, github desktop) and want to keep autofix
  1. Run git pull. this will merge the automated commit into your local copy of the PR branch.
  2. Continue working.
I do not want the automated commit
  1. Format your code locally, then commit it.
  2. Run git push --force to force push your branch. This will overwrite the automated commit on remote with your local one.
  3. Continue working.

If you don't do this, your following commits will be based on the old commit, and cause MERGE CONFLICT.

@chaosvolt
Copy link
Member

You could potentially also use impact fuzes (impact_fuze) in addition to or instead of regular fuzes for flavor.

@NerdPunkFu
Copy link
Contributor Author

You could potentially also use impact fuzes (impact_fuze) in addition to or instead of regular fuzes for flavor.

That probably wouldn't work actually. There are different impact fuzes but a lot of them are something like this. As you can see, the fuze relies on quite a bit of force and the inertia of the round. An anti-tank mine might work with something like that, but a human foot isn't likely to generate enough force to crush that fuze and send the firing pin into the initial charge(the safety could probably be circumvented easy enough).

@chaosvolt
Copy link
Member

Fair point, I just figured it'd be good for flavor, a better choice than gunpowder fuses (though having a craftable option is probably okay for balance).

chaosvolt
chaosvolt previously approved these changes Mar 25, 2024
@NerdPunkFu
Copy link
Contributor Author

Fair point, I just figured it'd be good for flavor, a better choice than gunpowder fuses (though having a craftable option is probably okay for balance).

This is an example of how you could do a gunpowder fuse for a mine(or grenade). As I laid out before, you'd need a method of lighting it though and maybe a spring mechanism because random amounts of pressure applied over a random amount of time is probably not the best way to strike a match or some such.

@scarf005 scarf005 changed the title fix: Recipe changes to address land mine component inconsistencies fix(balance): Recipe changes to address land mine component inconsistencies Mar 25, 2024
@Lamandus
Copy link
Contributor

You could potentially also use impact fuzes (impact_fuze) in addition to or instead of regular fuzes for flavor.

That probably wouldn't work actually. There are different impact fuzes but a lot of them are something like this. As you can see, the fuze relies on quite a bit of force and the inertia of the round. An anti-tank mine might work with something like that, but a human foot isn't likely to generate enough force to crush that fuze and send the firing pin into the initial charge(the safety could probably be circumvented easy enough).

on the one hand you are right,

on the other, I would like to have the chance to use impact fuses, too, we can craft high end pneumatic weaponary with very ordinary things, so I think making it work with a impact fuse, would be favourable.

Copy link
Contributor

@Lamandus Lamandus left a comment

Choose a reason for hiding this comment

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

Adding the other fuse would be better.

@chaosvolt chaosvolt dismissed their stale review March 26, 2024 15:29

I guess sure then, dew eet

@NerdPunkFu
Copy link
Contributor Author

Adding the other fuse would be better.

Added it

Lamandus
Lamandus previously approved these changes Mar 26, 2024
@chaosvolt chaosvolt merged commit f9353aa into cataclysmbnteam:main Mar 26, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JSON related to game datas in JSON format.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Land mines explosive content inconsistent with power and weight compared to grenades and real life equivalents
3 participants