-
Notifications
You must be signed in to change notification settings - Fork 4.3k
Add radishes and seeds to the game #79490
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
Conversation
Spell checker encountered unrecognized words in the in-game text added in this pull request. See below for details. Click to expand
This alert is automatically generated. You can simply disregard if this is inaccurate, or (optionally) you can also add the new words to Hints for adding a new word to the dictionary
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider adding the taproot to cooking_components.json
where appropriate.
Separately, if you plant the radish seeds, you will then be able to harvest some taproots but not new seeds, making the seeds non-renewable. Do I understand this right? (this is just personal curiosity really)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're also not adding them to the vegetable requirement group, so they'll be unusable in most crafting recipes. Add them there.
I think you have to leave them to grow longer to get useful seeds https://en.wikipedia.org/wiki/Radish#Seed_pod_varieties so maybe just put a couple in as a byproduct? |
Oh, so that's different from carrots, cabbages, turnips, onions and many other "similar" vegetables that are biennial (i.e. producing seeds on the second year). Yeah, having seeds as a byproduct is not entirely unreasonable here then! (but it's probably also okay to leave them be nonrenewable given that other biennial plants are like that(?) already) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not checking if the values are accurate to real life but the JSON at least seems fine now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable.
(There are no seed byproducts but that's fine with me. It would be a bigger project anyway)
Summary
Content "Add radishes and seeds"
Purpose of change
Introduce radishes as a fairly quick going vegetable, which would be helpful in a survival scenario.
I believe they are available in New England per https://gobotany.nativeplanttrust.org/species/raphanus/sativus/
Describe the solution
Added seeds and taproots using Wikipedia nutrient information. Updated some itemgroups and spawns to include, at a lower probability as I think (I'm not a native of the USA) they might be less common than other sorts of veg.
Describe alternatives you've considered
Testing
Spawn seeds and taproots. Check planting. Check recipes allow crafting using radishes.
Additional context
(I made the mistake of initially creating a branch from another branch, not master, but I think i rebase'd and tidied it up, but some unrelated stuff shows in the changelog)