Skip to content

Commit 12ff785

Browse files
committed
yield calc fixes for sub recipe optimization
1 parent 9ecc17d commit 12ff785

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

Classes/ResultData.lua

+3-2
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ function CraftSim.ResultData:IsMinimumQualityReachable(qualityID)
101101

102102
local reachable = qualityID <= self.expectedQualityConcentration
103103
local concentrationOnly = self.expectedQuality < self.expectedQualityConcentration and
104-
qualityID == self.expectedQualityConcentration
104+
qualityID == self.expectedQualityConcentration
105105

106106
return reachable, concentrationOnly
107107
end
@@ -138,7 +138,8 @@ function CraftSim.ResultData:Update()
138138

139139
local professionStats = self.recipeData.professionStats
140140

141-
self.expectedYieldPerCraft = CraftSim.CALC:GetExpectedItemAmountMulticraft(recipeData)
141+
self.expectedYieldPerCraft = self.recipeData.baseItemAmount +
142+
(select(2, CraftSim.CALC:GetExpectedItemAmountMulticraft(recipeData)) * professionStats.multicraft:GetPercent(true))
142143

143144
-- special case for no quality results. Needed for expectedCrafts and such
144145
if not recipeData.supportsQualities then

CraftSim.toc

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
## Title: CraftSim
55
## Notes: Calculates the average profit based on your profession stats and other tools for dragonflight gold making
66
## Author: genju
7-
## Version: 17.1.8
7+
## Version: 17.1.9
88
## X-Curse-Project-ID: 705015
99
## X-Wago-ID: 0mNwaPKo
1010
## X-WoWI-ID: 26519

Data/News.lua

+3
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,9 @@ function CraftSim.NEWS:GET_NEWS(itemMap)
1515
local news = {
1616
f.bb(" Hello and thank you for using CraftSim!\n"),
1717
f.bb(" ( You are awesome! )"),
18+
newP("17.1.9"),
19+
f.p .. "Fixed Average Yield Calculation for Sub Recipe Optimization",
20+
f.a .. "- Thank you " .. f.bb("DiegoSnoop") .. " for pointing that out!",
1821
newP("17.1.8"),
1922
f.p .. "Fixed concentration calculation not considering",
2023
f.a .. "less concentration usage from specs",

0 commit comments

Comments
 (0)