CRITICAL: Double gravity applied to player ship each frame #3

New issue

Closed

opened 2026-05-11 06:16:42 +00:00 by Mark · 0 comments

Mark commented

2026-05-11 06:16:42 +00:00

Owner

Description

Both Ship.update() and PhysicsEngine.update() apply gravitational forces to the player ship each frame, resulting in double gravity.

Root cause

In main.js:

Line 262: this.playerShip.update(dt, wells, sl) -- Ship.update() lines 115-124 apply gravity from wells to velocity
Line 267: this.physics?.update(this.playerShip, ...) -- PhysicsEngine.update() lines 222-227 ALSO apply gravity from wells to velocity

Impact

Ships experience approximately 2x expected gravitational pull, making orbits unstable and gameplay feel wrong.

Fix

Remove gravity integration from one of the two methods. Ship.update() should handle ship-internal physics (thrust, RCS, resources) while PhysicsEngine.update() handles external forces (gravity, collisions). Or vice-versa.

## Description Both `Ship.update()` and `PhysicsEngine.update()` apply gravitational forces to the player ship each frame, resulting in double gravity. ## Root cause In `main.js`: - Line 262: `this.playerShip.update(dt, wells, sl)` -- Ship.update() lines 115-124 apply gravity from wells to velocity - Line 267: `this.physics?.update(this.playerShip, ...)` -- PhysicsEngine.update() lines 222-227 ALSO apply gravity from wells to velocity ## Impact Ships experience approximately 2x expected gravitational pull, making orbits unstable and gameplay feel wrong. ## Fix Remove gravity integration from one of the two methods. Ship.update() should handle ship-internal physics (thrust, RCS, resources) while PhysicsEngine.update() handles external forces (gravity, collisions). Or vice-versa.