fix(formula): calculate performance

dream-num · Oct 21, 2024 · 50dc1d2 · 50dc1d2
1 parent 5385ee6
commit 50dc1d2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/packages/engine-formula/src/commands/mutations/set-formula-calculation.mutation.ts b/packages/engine-formula/src/commands/mutations/set-formula-calculation.mutation.ts
@@ -45,9 +45,7 @@ export interface ISetFormulaCalculationNotificationMutation {
 export const SetFormulaCalculationNotificationMutation: IMutation<ISetFormulaCalculationNotificationMutation> = {
     id: 'formula.mutation.set-formula-calculation-notification',
     type: CommandType.MUTATION,
-    handler: () => {
-        return true;
-    },
+    handler: () => true,
 };
 
 export interface ISetFormulaCalculationResultMutation {

diff --git a/packages/sheets-formula/src/controllers/trigger-calculation.controller.ts b/packages/sheets-formula/src/controllers/trigger-calculation.controller.ts
@@ -203,8 +203,6 @@ export class TriggerCalculationController extends Disposable {
 
                     this._commandService.executeCommand(SetFormulaCalculationStartMutation.id, { ...this._executingDirtyData }, lo);
 
-                    this._startExecutionTime = performance.now();
-
                     this._waitingCommandQueue = [];
                 }, 100);
             })
@@ -341,6 +339,8 @@ export class TriggerCalculationController extends Disposable {
                     const { startCalculate, formulaCount } = command.params as ISetFormulaCalculationNotificationMutation;
 
                     if (startCalculate) {
+                        this._startExecutionTime = performance.now();
+
                         // Increment the calculation process count and assign a new ID
                         calculationProcessCount++;