Thorinair / Stardew-Profits

A Stardew Valley crop profits calculator and visualizer. Link: https://thorinair.github.io/Stardew-Profits/
MIT License
153 stars 74 forks source link

[Feature] Dehydrator Product Option #69

Closed doodlebunnyhops closed 1 month ago

doodlebunnyhops commented 2 months ago

Duplicate of #59 apologies I didn't have access to the repo

Feature Added:

Notable Mentions

Thorinair commented 1 month ago

This looks very good! I will be merging it with dev branch later today and having a look at everything. Thanks!

Thorinair commented 1 month ago

I've merged and had a look at it, did some slight changes with how some of the parameters are named. I renamed "Non-existent" to "No Artisan Good", and renamed the new parameter to "Dehydrator Excess".

I noticed an issue however, which prevents me from pushing i to master. Firstly, I think that the processing should work per harvest and not for the total crops per season. A lot of other features work like this, like the Equipment count for instance, so it would make sense. This would mean that you need to have at least 5 crops planted for some of the crops, and that is okay. Further more, the dehydrator calculations seem to ignore the need of keeping half the produce for seeds, if Process & Replant is enabled.

Would you please be able to pull dev again, and implement some of these changes as a new commit? I would really appreciate it!

doodlebunnyhops commented 1 month ago

Ah excellent point per harvest ;) happy to update!