"Artisan" skill is seemingly not used in calculation for profits at all.

Thorinair / Stardew-Profits

A Stardew Valley crop profits calculator and visualizer. Link: https://thorinair.github.io/Stardew-Profits/

MIT License

163 stars 79 forks source link

"Artisan" skill is seemingly not used in calculation for profits at all. #77

Closed WyattAdams closed 3 months ago

WyattAdams commented 3 months ago

Screenshot 2024-06-14 161514 (Image without artisan selected)

While looking at profits for strawberry wine, I've noticed that the numbers are entirely unaffected across the board whether or not you have the artisan perk selected, when the numbers should be increased (as far as gross profit anyways) by a direct 40% for all wine/artisan good sales.

Screenshot 2024-06-14 161524 (Image with artisan selected)

doodlebunnyhops commented 3 months ago

submitted fix for review: PR #78

Thorinair commented 3 months ago

It is now fixed and released on the public version! Big oops on my part! xD Thanks @doodlebunnyhops for quickly submitting a PR for the fix! :)

doodlebunnyhops commented 3 months ago

😅 my whoopsie too - I hadn't done a check after the big dehydrator work but glad to help!