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

Raise spawn chance for cardboard boxes. #35876

Merged
merged 3 commits into from
Dec 6, 2019
Merged

Raise spawn chance for cardboard boxes. #35876

merged 3 commits into from
Dec 6, 2019

Conversation

arijust
Copy link
Contributor

@arijust arijust commented Dec 5, 2019

Summary

SUMMARY: Balance "Raise spawn chance for cardboard boxes."

Purpose of change

Close #35795

Describe the solution

Tweak probability to get boxes in several item groups.

Describe alternatives you've considered

Raise spawn chances a bit more and make some items spawn in medium boxes.

@ZhilkinSerg ZhilkinSerg added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Game: Balance Balancing of (existing) in-game features. Items / Item Actions / Item Qualities Items and how they work and interact Spawn Creatures, items, vehicles, locations appearing on map labels Dec 5, 2019
Copy link
Member

@I-am-Erk I-am-Erk left a comment

Choose a reason for hiding this comment

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

Glad to see someone else giving cardboard a little love.

For the non-trash item groups, like hardware, you want to make boxes into a separate collection that can appear alongside other goods. Otherwise all you're doing is increasing the chance that instead of a useful item, the player just finds an empty cardboard box, effectively just decreasing the spawn of tools and things in favour of cardboard. Dino Dave would approve, but I don't think it's what you're going for.

I'll show you what I mean as soon as I'm off my phone. Collections are a powerful tool for making our item groups smarter, and we need more people to start using them on projects like this.

@ZhilkinSerg ZhilkinSerg merged commit 4b974f2 into CleverRaven:master Dec 6, 2019
@SpellingSword
Copy link

Thanks :)

@I-am-Erk
Copy link
Member

I-am-Erk commented Dec 6, 2019

@ZhilkinSerg I was kinda in the midst of suggesting changes there...

@I-am-Erk
Copy link
Member

I-am-Erk commented Dec 6, 2019

So @arijust I can go do it in a separate pr, but if you'd like to learn to adjust this stuff let me know either here or on the discord. It's not incredibly difficult.

Basically the code looks like this

{
        "collection": [
          { "item": "plastic_spoon", "count": [ 0, 6 ] },
          { "item": "plastic_knife", "count": [ 0, 6 ] },
          { "item": "plastic_fork", "count": [ 0, 6 ] }
        ],
        "prob": 30
      },
      { "item": "plastic_spoon_kids", "prob": 35, "count": [ 1, 6 ] },
      [ "corkscrew", 75 ],

Everything inside the added collection has a chance of spawning together, so if you made that eg. a medium cardboard box and some packs of nails, it would spawn all those things together, implying a cardboard box full of packs of nails. It's not quite a container because our code isn't there yet, but it's a decent functional representation of one.

@arijust
Copy link
Contributor Author

arijust commented Dec 6, 2019

I will note that for my future PRs, but it would take me some time before I get used to it.

@I-am-Erk
Copy link
Member

I-am-Erk commented Dec 6, 2019

Yep no worries. I will probably make a second PR adjusting the changes in this one so you can see what I mean, and then if you want to further add cardboard boxes to spawn along with things that might spawn with boxes you'll be able to see how. TOGETHER, WE COULD FILL THE EARTH WITH CARDBOARD.

@arijust
Copy link
Contributor Author

arijust commented Dec 6, 2019

Alright, thanks.

@arijust arijust deleted the cardboard_box_buff branch December 6, 2019 17:23
I-am-Erk pushed a commit to I-am-Erk/Cataclysm-DDA that referenced this pull request Dec 6, 2019
* Raise chance to get boxes in supplies_hardware.

* Raise change for medium_box in mischw group.

* Raise chances to get boxes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Game: Balance Balancing of (existing) in-game features. Items / Item Actions / Item Qualities Items and how they work and interact [JSON] Changes (can be) made in JSON Spawn Creatures, items, vehicles, locations appearing on map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Normal" cardboard boxes are absurdly rare
4 participants