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

"Pay For Seeds" estimates seed cost improperly for plants which have multiple harvests #91

Open
americobarros opened this issue Nov 15, 2024 · 1 comment

Comments

@americobarros
Copy link

americobarros commented Nov 15, 2024

Love this tool!

Edit: See comments for updated issue.

I noticed though that the "Pay For Seeds" option seems to use the quantity sold to calculate seed cost, rather than the number planted. This causes an over estimation of seed cost.

For example, below I said that I had 16 crops of pumpkins planted on the first day. This would result in needing 16 pumpkin seeds (since they can be harvested multiple times) and would produce 32 harvests/pumpkins.
At $100 per seed, the total cost of the seeds would be $1600. Instead, it is calculated as $3200. Heres a link to the full results
image

@americobarros
Copy link
Author

Sooo I totally read the wiki wrong, and even tho it looks like it, pumpkins do NOT regrow 🤦
The seed cost is calculated using whether or not they regrow however, there IS a discrepancy between the Planted variable and the Seed Loss variable which caused me to make this error.

Seed Loss assumes you plant 16 crops, and then replant them before the end of the month using new seeds.
32 seeds x 100 = $3200
Planted seems to only use the Number of Crops -> therefore 16 seeds
16 seeds x 100 = $1600

Replanting new seeds should be used to calculate the number planted.
Clarity wise, the Planted variable could be changed to seeds needed for clarity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant