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

remove buckwheat from foraging, add to seed group #77912

Merged
merged 1 commit into from
Nov 17, 2024

Conversation

NetSysFire
Copy link
Member

Summary

Content "Buckwheat can no longer be foraged but may still be found where seeds spawn"

Purpose of change

I found a lot of buckwheat while foraging ingame and this got me curious to do research.

The result is: This commercially-grown buckwheat is absolutely not native to Massachusetts. These are like the peanut bushes we had from the whales era: they show up for groundnuts when you google them, but are absolutely not those groundnuts. And this "buckwheat" is Fallopia convolvulus, black bindweed. I did a lot of research the past two hours and the result of that is that some bindweed species were cultivated by native peoples on the great plains and black bindweed is an ancient crop from the bronze age.

Black bindweed has tiny seeds and is very fiddly to work with it appears. Plus there is absolutely no nutritional info, so implementing this ingame would take a lot of guessing for something as small and irrelevant as black bindweed.

Describe the solution

Can not be foraged anymore but can be found in the farming seeds group. This officially turns it into the cultivated variety, exclusively. One godco quest uses buckwheat, so this is why I moved it into the seed group. This will make the quest considerably harder and maybe its better to replace buckwheat in that quest with another crop like wheat or even the native ground nuts.

Describe alternatives you've considered

I looked at getting rid of buckwheat altogether but that would be very complex and buckwheat is a plausible commercial crop to appear and it has many uses. So the wild bindweed here is the odd one out.

Testing

None needed, this just modifies weighted lists. Only consideration is balance, which I explained above.

Additional context

Sources:

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Spawn Creatures, items, vehicles, locations appearing on map astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Nov 16, 2024
@Maleclypse Maleclypse merged commit aa5f97a into CleverRaven:master Nov 17, 2024
20 of 26 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 [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Spawn Creatures, items, vehicles, locations appearing on map
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants