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

Mil bunker gunmods part 2 #2581

Merged

Conversation

metalheimer
Copy link
Contributor

@metalheimer metalheimer commented Apr 7, 2023

Summary

SUMMARY: [Balance] "Military bunker gun mod spawns attempt 2"

Purpose of change

Increased spawns for one of the bunker variants (the autodoc one). I was having difficulty getting spawns there. The curse of item spawns in this game is there are so many steps/filters to pass. You have your own selected item spawn rate, but then under the hood there are further probabilities defined in the files to pass before you get a spawn. Double or triple layers of probabilities, often grinding the spawn chances down to small numbers.

Also removed shoulder strap from the sopmod_2 spawn list. Considered removing all the other craftable gun mods but didn't. Might feel cute and delete them later.

Describe the solution

With this update, the autodoc mil bunker variant now has, or at least should have 100% chance of spawning 2-3 mods as long as your world's item drop rate is 1.00. Doesn't mean they're the cool gun mods. See? Another hurdle to pass to juicy item spawns. This is why you never see the cool stuff in the game. Because you live sedentary lifestyle even in the game, consuming only the resources in your vicinity, only moving on when there's nothing left to do. You overthink, overanalyze and hesitate. You want to be efficient, almost perfectionist, but end up being not so efficient. Do you know what this is called? Projection.

Describe alternatives you've considered

Therapy.

Testing

Spawned a bunch of bunkers and checked.

Additional context

If you want to see these changes in your existing game, you'll possibly first have to change (increase) the item spawn rate to 1.00 for your current world, then find a bunker that's far enough away, possibly in the next overmap.

Forgot: No Hope mod possibly needs to be edited too because it changes the bunker.json. If you play with No Hope enabled, these changes don't take effect. No gun mods for you, you hopeless wretch without hope.

@github-actions github-actions bot added the JSON related to game datas in JSON format. label Apr 7, 2023
@Firestorm01X2
Copy link
Collaborator

Firestorm01X2 commented Apr 7, 2023

Just remainder - if your PR is not ready to merge and you want to work with it for some time, then you can set [WIP] before name of your PR or convert PR to draft:

изображение

That will inform reviewer that work is in progress. It is very useful, since sometime someone may merge your PR accendialy before it is ready.

@scarf005
Copy link
Member

scarf005 commented Apr 7, 2023

image

also, adding unseparated hash tag with number references other issues/PRs/discussions, if you want to convey it's second attempt (rather than linking PR #2 to this PR), you may want to change your PR title to something like part 2.

@scarf005 scarf005 changed the title Mil bunker gunmods #2 Mil bunker gunmods part 2 Apr 7, 2023
@scarf005 scarf005 marked this pull request as draft April 8, 2023 01:52
@scarf005
Copy link
Member

scarf005 commented Apr 8, 2023

Forgot: No Hope mod possibly needs to be edited too because it changes the bunker.json. If you play with No Hope enabled, these changes don't take effect. No gun mods for you, you hopeless wretch without hope.

i've converted it in draft for you, please change back to read for review if you're finished.

@metalheimer metalheimer marked this pull request as ready for review April 8, 2023 17:02
@Firestorm01X2 Firestorm01X2 merged commit 2f20b7d into cataclysmbnteam:upload Apr 9, 2023
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.

3 participants