Thorinair / Stardew-Profits

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

Average profits per day wrong for cross season crops #18

Closed Azzinoth224 closed 3 years ago

Azzinoth224 commented 5 years ago

The average profits per day calculation is wrong for cross season crops. It does correctly calculate the profits over multiple seasons, bet then divides only by 28 days, instead of 28 days times number of seasons. For example for coffe beans: It calculates correctly 23 harvests (9 harvests in Spring and 14 in Summer) and calculates the profits for that. But it only shows a duration of 28 days in the tooltip (it should be 56 days) and it also uses 28 days for the average per day calculation, which means that value is by a factor 2 too big.