Skip to content

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

Merged
merged 13 commits into from
Feb 3, 2025
Merged

Conversation

juur
Copy link
Contributor

@juur juur commented Feb 2, 2025

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)

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display Items: Food / Vitamins Comestibles and drinks Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Spawn Creatures, items, vehicles, locations appearing on map json-styled JSON lint passed, label assigned by github actions labels Feb 2, 2025
Copy link
Contributor

github-actions bot commented Feb 2, 2025

Spell checker encountered unrecognized words in the in-game text added in this pull request. See below for details.

Click to expand
  • Crunchy red radish raproots

This alert is automatically generated. You can simply disregard if this is inaccurate, or (optionally) you can also add the new words to tools/spell_checker/dictionary.txt so they will not trigger an alert next time.

Hints for adding a new word to the dictionary
  • If the word is normally in all lowercase, such as the noun word or the verb does, add it in its lower-case form; if the word is a proper noun, such as the surname George, add it in its initial-caps form; if the word is an acronym or has special letter case, such as the acronym CDDA or the unit mW, add it by preserving the case of all the letters. A word in the dictionary will also match its initial-caps form (if the word is in all lowercase) and all-uppercase form, so a word should be added to the dictionary in its normal letter case even if used in a different letter case in a sentence.
  • For a word to be added to the dictionary, it should either be a real, properly-spelled modern American English word, a foreign loan word (including romanized foreign names), or a foreign or made-up word that is used consistently and commonly enough in the game. Intentional misspelling (including eye dialect) of a word should not be added unless it has become a common terminology in the game, because while someone may have a legitimate use for it, another person may spell it that way accidentally.

@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Feb 2, 2025
Copy link
Contributor

@moxian moxian left a 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)

Copy link
Contributor

@Karol1223 Karol1223 left a 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.

@juur
Copy link
Contributor Author

juur commented Feb 2, 2025

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)

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?

@moxian
Copy link
Contributor

moxian commented Feb 2, 2025

The seeds of radishes grow in siliques (widely referred to as "pods"), following flowering that happens when left to grow past their normal harvesting period.

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)

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 2, 2025
@juur juur requested a review from Karol1223 February 2, 2025 11:10
@github-actions github-actions bot removed the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 2, 2025
Copy link
Contributor

@Karol1223 Karol1223 left a 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

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 2, 2025
@juur juur requested a review from moxian February 2, 2025 14:14
Copy link
Contributor

@moxian moxian left a 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)

@Maleclypse Maleclypse merged commit 1b8b7d5 into CleverRaven:master Feb 3, 2025
23 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Items: Food / Vitamins Comestibles and drinks [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display Spawn Creatures, items, vehicles, locations appearing on map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants