From 587d5aa19d3d7b8305d28e682c640e3555bdb9dc Mon Sep 17 00:00:00 2001 From: Ean Milligan Date: Wed, 6 Aug 2025 14:30:41 -0400 Subject: [PATCH] -1 loop per iteration by only doing rollDist when needed --- src/artigen/artigen.ts | 2 +- src/artigen/managers/handler/workerComplete.ts | 2 +- src/artigen/math/mathTokenizer.ts | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/artigen/artigen.ts b/src/artigen/artigen.ts index 99b7c96..1c0d40f 100644 --- a/src/artigen/artigen.ts +++ b/src/artigen/artigen.ts @@ -170,7 +170,7 @@ export const runCmd = (rollRequest: QueuedRoll): SolvedRoll => { returnMsg.line3 = line3; // Reduce rollDist maps into a single map - returnMsg.rollDistributions = reduceRollDistMaps(tempRollDists); + if (rollRequest.modifiers.rollDist) returnMsg.rollDistributions = reduceRollDistMaps(tempRollDists); } catch (e) { // Fill in the return block const solverError = e as Error; diff --git a/src/artigen/managers/handler/workerComplete.ts b/src/artigen/managers/handler/workerComplete.ts index 78f7b58..ca8fc7c 100644 --- a/src/artigen/managers/handler/workerComplete.ts +++ b/src/artigen/managers/handler/workerComplete.ts @@ -278,7 +278,7 @@ Please click on "<@${botId}> *Click to see attachment*" above this message to se details: returnMsg.line3, }, counts: rollRequest.modifiers.count ? returnMsg.counts : null, - rollDistributions: returnMsg.rollDistributions.entries().toArray(), + rollDistributions: rollRequest.modifiers.rollDist ? returnMsg.rollDistributions.entries().toArray() : null, }, }), { diff --git a/src/artigen/math/mathTokenizer.ts b/src/artigen/math/mathTokenizer.ts index c7361af..497de35 100644 --- a/src/artigen/math/mathTokenizer.ts +++ b/src/artigen/math/mathTokenizer.ts @@ -201,7 +201,7 @@ export const tokenizeMath = ( const formattedRoll = formatRoll(executedRoll, modifiers); mathConf[i] = formattedRoll.solvedStep; countDetails.push(formattedRoll.countDetails); - rollDists.push(formattedRoll.rollDistributions); + if (modifiers.rollDist) rollDists.push(formattedRoll.rollDistributions); } } @@ -318,7 +318,7 @@ export const tokenizeMath = ( const formattedRoll = formatRoll(executedRoll, modifiers); mathConf[rollGroupIdx] = formattedRoll.solvedStep; countDetails.push(formattedRoll.countDetails); - rollDists.push(formattedRoll.rollDistributions); + if (modifiers.rollDist) rollDists.push(formattedRoll.rollDistributions); }); }