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

Better Bounties #9381

Merged
merged 7 commits into from
Jul 21, 2020
Merged

Better Bounties #9381

merged 7 commits into from
Jul 21, 2020

Conversation

Doxxmedearly
Copy link
Contributor

  • rscadd: "Added new cargo bounties."
  • rscadd: "Adjusted cargo bounties. Reward amounts are semi-randomized, as is the number of required items for some bounties. The types of bounties that appear are more randomized."
  • tweak: "Altered some bounty descriptions."
  • tweak: "Any food that has bites taken out of it will no longer count for the related bounty."
  • tweak: "More types of pies, muffins, and kabobs should now count for their respective bounties."
  • tweak: "Further adjusted cargo warehouse spawns."

Alright there's a bit going on here.

New bounties- self explanatory. We need more variety, especially since the PR that (rightfully) weeded out most of the meme ones. This includes wholly new bounties, new reagent types, and even a few different sections- engineering in particular was left out by bounties.

Semi-randomization: Just to add a bit of variety, there's some randomization. Reward amounts are picked from a range, so it's not always just one flat amount. If a bounty requires multiple items (ie, 6 toolboxes), there's some randomization, too. Rewards will be adjusted by this; if you need to collect more than the required_count, the reward will increase.

Also it was kind of super predictable that you'd ALWAYS have x types of bounties. I've only very slightly adjusted this to provide a bit more variety.

descriptions: self explanatory.

food bounty stuff: self explanatory. i fucking hate food typepaths.

warehouse spawns: just some further adjustments after some in-round testing. NVGs and jammers were appearing far too frequently. fire extinguishers were spammed, and I added just a bit more variety.

@Doxxmedearly Doxxmedearly requested a review from Arrow768 as a code owner July 18, 2020 01:02
@Doxxmedearly
Copy link
Contributor Author

!review

@alsoandanswer
Copy link
Contributor

alsoandanswer commented Jul 18, 2020

IMHO, prices are far too low.
For starters, an organ...
is the same price as a cup of ramen.
I suggest to keep the prices in the 1000 and above range as a benchmark.
Some of these are horrendously low - lungs...for 35 credits. Cup ramen is 20 credits.

In-game, credits are pretty much worthless, and this'll create even less incentive to actually do them.
Which is pretty bad, considering they aren't really done at all in the first place.
TL;DR - raise the prices.

Good work on increasing the variety of food items accepted though.

@Doxxmedearly
Copy link
Contributor Author

IMHO, prices are far too low.
For starters, an organ...
is the same price as a cup of ramen.
I suggest to keep the prices in the 1000 and above range as a benchmark.
Some of these are horrendously low - lungs...for 35 credits. Cup ramen is 20 credits.

In-game, credits are pretty much worthless, and this'll create even less incentive to actually do them.
Which is pretty bad, considering they aren't really done at all in the first place.
TL;DR - raise the prices.

Good work on increasing the variety of food items accepted though.

as the new vars in the /datum/bounty state, the amounts are multiplied by 100. 25 is 2500. This is because if I just did rand(2000, 3000) we'd get very sloppy numbers like 2743. This keeps it tidy.

@mikomyazaki
Copy link
Contributor

IMHO, prices are far too low.
For starters, an organ...
is the same price as a cup of ramen.
I suggest to keep the prices in the 1000 and above range as a benchmark.
Some of these are horrendously low - lungs...for 35 credits. Cup ramen is 20 credits.
In-game, credits are pretty much worthless, and this'll create even less incentive to actually do them.
Which is pretty bad, considering they aren't really done at all in the first place.
TL;DR - raise the prices.
Good work on increasing the variety of food items accepted though.

as the new vars in the /datum/bounty state, the amounts are multiplied by 100. 25 is 2500. This is because if I just did rand(2000, 3000) we'd get very sloppy numbers like 2743. This keeps it tidy.

Another way to do round numbers would be to just do... round(price, 100) and round to the nearest 100. The way you've done it works, although it's a bit less clear.

@Doxxmedearly
Copy link
Contributor Author

IMHO, prices are far too low.
For starters, an organ...
is the same price as a cup of ramen.
I suggest to keep the prices in the 1000 and above range as a benchmark.
Some of these are horrendously low - lungs...for 35 credits. Cup ramen is 20 credits.
In-game, credits are pretty much worthless, and this'll create even less incentive to actually do them.
Which is pretty bad, considering they aren't really done at all in the first place.
TL;DR - raise the prices.
Good work on increasing the variety of food items accepted though.

as the new vars in the /datum/bounty state, the amounts are multiplied by 100. 25 is 2500. This is because if I just did rand(2000, 3000) we'd get very sloppy numbers like 2743. This keeps it tidy.

Another way to do round numbers would be to just do... round(price, 100) and round to the nearest 100. The way you've done it works, although it's a bit less clear.

Yeah i thought about that like a day after i uploaded it. Ugh.

@Doxxmedearly
Copy link
Contributor Author

No point is leaving in confusing shitcode. Changed it to be more readable.

@BotBOREALIS BotBOREALIS added the Waiting Period The PR has the required number of reviews, but is still in the waiting period label Jul 20, 2020
Co-authored-by: Geeves <ggrobler447@gmail.com>
@Doxxmedearly
Copy link
Contributor Author

someone kick travis please

@BotBOREALIS BotBOREALIS added Awaiting Merge and removed Review Required Waiting Period The PR has the required number of reviews, but is still in the waiting period labels Jul 21, 2020
@skull132 skull132 merged commit a23d941 into Aurorastation:master Jul 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants