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

Cargo tweaks & rebalances + New Dialogue #4392

Merged
merged 14 commits into from
May 1, 2021
Merged

Cargo tweaks & rebalances + New Dialogue #4392

merged 14 commits into from
May 1, 2021

Conversation

Momoberry
Copy link
Contributor

@Momoberry Momoberry commented Apr 20, 2021

About the PR

*Changes several minimum/maximum values for several commodities being sold/purchased through QM Console. Increases the Buy_Cap from 20 to 99 (99 items + Invoice). Changes around some values for # of items sold from traders. Ex. Gasmasks from 10 units to 25 units. Notable changes are nerfing cargo holds and jumpsuits and increasing other items slightly/moderately to bring more options to make money now that buy limit has increased. Gragg's starstone is now from 500,000-4,500,000 instead of a static 3,000,000. Let me know if this change isn't liked, as I put it in to bring in some variety to how it operates mostly. Also, let me know if 99 seems too high, I tested it and it seemed fine but you can't easily get every scenario tested since much of cargo is based off chance. If 99 does end up causing too many issues, reducing it to 50 was the next plan.

Adds several new dialogue lines to generic traders. Brought up general hike tolerance to 25 from 20 for clarity. Tweaks several duration times in Cargo, such as the supply doors and the QR Code Printer (SFX have also been adjusted). Also tweaks the chances for generic traders to arrive and leave, as well as increasing their maximum sold items to 4 instead of 3.

Changed the size of Supply Records PDA File from 8 to 6. QMs are one of the only jobs that regularly use their PDAs, so this just cleans up and gives them a bit more room to work with.

Also changes the artifact resupply timer from 3-8 minutes to 1-5 minutes, after an artifact is properly labelled and sold. ALSO removes the research cardlock from the chemistry resupply crate, as there really isn't anything remotely dangerous in there. Also cargo could really use access to beakers, honestly.

Why's this needed?

Cargo currently has a bit of a bloat issue with the amount of crates coming in that only contain a small amount of items. Since disassembling crates isn't affected by carpenter and action bars limit the amount of crates to one at a time, if you have mass orders coming in, it can quickly get out of hand or clog up.

Cargo "meta" only used jumpsuits/food/cargo holds to make money as all the others had way too much interaction with the QM console to the point it was tedious, so the most common ways (cargo holds and jumpsuits) have been tuned down slightly and many of the other options like ore, herbs, electronics, and a few others have been increased accordingly to match up and become valid ways to make money for the budget. Since this change increase the buy cap to 99, people may ACTUALLY want to buy the construction materials/fabrics and other materials sold at an unlimited amount like char. Which will IDEALLY bring a bit more interaction between departments, as well as just cut down on the amount of web browsing and mindless clicking & crate hell.

The chance changes are to make cargo less reliant on a single trader to make their entire budget from and then do nothing for the rest of the round. It gives them a lot more selection to choose from, but also the increased chance to leave will hopefully start steering people away from buying ten "200 metal sheets" crates at round start and instantly crafting pod parts.

As for the dialogue changes, they'll ideally bring a bit more variation and character into the masked man/woman/martian that is the generic trader, instead of just seeing the same lines over and over again. Besides, more variety never hurts! *Would really appreciate feedback and suggestions on the current dialogue changes and for future additions.

Changelog

(u)MomoBerry
(*)Changes several values for sold and bought commodities, and increases the QM console buy cap from 20 to 99. New dialogue for traders, yay personality!
(+)Removes Research Cardlock from "Chemistry Resupply" Crate.

Momoberry and others added 7 commits April 20, 2021 01:40
* Rocko so silly

* Hats!

* Over 9000?

* Rip Out Fancy Spancy Stuff... until I can get a Dev approval

* Overlays with additive or subtractive blending no get flat good

* Give Rocko a Home...

* Addressing code review comments

* Get Rocko out of the crate! Silly rock.

* Clarion too!
@boring-cyborg boring-cyborg bot added the C-Sound Automatically applied on any .ogg or sound folder change. label Apr 20, 2021
@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 20, 2021
code/datums/supply_packs.dm Show resolved Hide resolved
@@ -8,7 +8,7 @@
var/time_until_shift = 0.0
var/demand_multiplier = 2
var/list/active_traders = list()
var/max_buy_items_at_once = 20
var/max_buy_items_at_once = 99
Copy link
Contributor

@Carbadox Carbadox Apr 20, 2021

Choose a reason for hiding this comment

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

this will have some considerable balance implications, 50 would be more reasonable

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I was going back and forth myself. 99 is a traditional value for the "max" to buy in games, plus it fits well with the 99 + invoice for max crate capacity. But, it could have balance issues like you said, but that's why I nerfed some of the things using unlimited purchase mats like fabric, etc.

code/modules/economy/traders/generic.dm Show resolved Hide resolved
code/modules/economy/traders/gragg.dm Show resolved Hide resolved
code/obj/machinery/computer/QM_supply.dm Show resolved Hide resolved
@keywordlabeler keywordlabeler bot added the E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. label Apr 20, 2021
@github-actions
Copy link
Contributor

Created discussion thread: https://forum.ss13.co/showthread.php?tid=16238

@keywordlabeler keywordlabeler bot added C-QoL A quality of life improvement that makes the game easier to play C-Balance Balance changes, buffs and nerfs labels Apr 20, 2021
@ghost
Copy link

ghost commented Apr 21, 2021

This is a suggestion, not a "gotta change it" thing, but if you're adding a whole new sound file, consider giving it a completely different sound profile to make it pop audibly rather than just slightly editing a current file. Giving the printer its own character via audio contributes positively to the soundscape. Example: if you have something using printer_thermal.ogg next to the thing using printer_cargo.ogg, you should be able to identify which device is printing based upon the sound alone. The slightly edited sound file doesn't really allow for that.

@Studenterhue Studenterhue added the E-Add-To-Wiki A PR that will require changes to the wiki label Apr 22, 2021
@Tarmunora
Copy link
Member

Hm. Would it be feasible to make traders request a certain number of an item(s), and then pay out a single bulk sum when the order is filled? Filling specific orders might be more interesting QM gameplay, and would be mechanically different than just shipping goods out to the general market

@ghost ghost requested a review from ZeWaka May 1, 2021 00:12
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

added the new printer audio, looks good to me, i'll hold off on merging until a coder takes a peek at the latest changes

@ZeWaka ZeWaka merged commit d69b85f into goonstation:master May 1, 2021
github-actions bot pushed a commit that referenced this pull request May 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Balance Balance changes, buffs and nerfs C-QoL A quality of life improvement that makes the game easier to play C-Sound Automatically applied on any .ogg or sound folder change. E-Add-To-Wiki A PR that will require changes to the wiki E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants