diff --git a/src/client/ClientGameRunner.ts b/src/client/ClientGameRunner.ts index e0804b885..348e80d6d 100644 --- a/src/client/ClientGameRunner.ts +++ b/src/client/ClientGameRunner.ts @@ -216,7 +216,6 @@ export class ClientGameRunner { private catchUpMode: boolean = false; private readonly CATCH_UP_ENTER_BACKLOG = 120; // turns behind to enter catch-up private readonly CATCH_UP_EXIT_BACKLOG = 20; // turns behind to exit catch-up - private readonly CATCH_UP_HEARTBEATS_PER_FRAME = 5; //upper bound on heartbeats per frame when in catch-up mode private pendingUpdates: GameUpdateViewData[] = []; private isProcessingUpdates = false; @@ -225,8 +224,7 @@ export class ClientGameRunner { private renderEveryN: number = 1; private renderSkipCounter: number = 0; private lastFrameTime: number = 0; - private readonly MAX_RENDER_EVERY_N = 60; - private lastBeatsPerFrame: number = 1; + private readonly MAX_RENDER_EVERY_N = 5; constructor( private lobby: LobbyConfig, @@ -317,7 +315,6 @@ export class ClientGameRunner { this.processPendingUpdates(); } }); - const worker = this.worker; const keepWorkerAlive = () => { if (this.isActive) { const now = performance.now(); @@ -327,14 +324,6 @@ export class ClientGameRunner { } this.lastFrameTime = now; - const beatsPerFrame = this.catchUpMode - ? this.CATCH_UP_HEARTBEATS_PER_FRAME - : 1; - this.lastBeatsPerFrame = beatsPerFrame; - for (let i = 0; i < beatsPerFrame; i++) { - worker.sendHeartbeat(); - } - // Decide whether to render (and thus process pending updates) this frame. let shouldRender = true; if (this.catchUpMode && this.renderEveryN > 1) { @@ -541,7 +530,6 @@ export class ClientGameRunner { this.backlogTurns, this.catchUpMode, this.renderEveryN, - this.lastBeatsPerFrame, ), ); diff --git a/src/client/graphics/layers/PerformanceOverlay.ts b/src/client/graphics/layers/PerformanceOverlay.ts index 3cd688449..bd101f80d 100644 --- a/src/client/graphics/layers/PerformanceOverlay.ts +++ b/src/client/graphics/layers/PerformanceOverlay.ts @@ -435,7 +435,7 @@ export class PerformanceOverlay extends LitElement implements Layer { private renderEveryN: number = 1; @state() - private beatsPerFrame: number = 1; + private beatsPerFrame: number | null = null; updateTickMetrics( tickExecutionDuration?: number, @@ -491,7 +491,7 @@ export class PerformanceOverlay extends LitElement implements Layer { this.renderEveryN = renderEveryN; } if (beatsPerFrame !== undefined) { - this.beatsPerFrame = beatsPerFrame; + this.beatsPerFrame = beatsPerFrame ?? null; } this.requestUpdate(); @@ -647,7 +647,8 @@ export class PerformanceOverlay extends LitElement implements Layer { ${this.inCatchUpMode ? html`