Opened 23 months ago

Closed 23 months ago

Last modified 23 months ago

#10041 closed defect (fixed)

TITANIC: Star Control: Forward and Backwords are too slow

Reported by: dafioram Owned by: dreammaster
Priority: low Component: Engine: Titanic
Keywords: Cc:
Game: Starship Titanic

Description

scummvm: f4f15667a5a7cdde12f5aa7ccf1b7384277bffc4

The movement forward and backwards of the starship using the keyboard keys ";" and "," was changed in:

a42172485efb47638bd4cb27f7be8d32247a5605

The keys go in the correct directions, but I believe backwards and forwards go way to slow. Also, the max speed may also be too low. I think the previous speed for reverse before this commit was probably ok.

Reverse is way too slow, you have to wait a few seconds to even tell if you are moving.

Change History (4)

comment:1 Changed 23 months ago by dafioram

I tried changing the max speed and change increment in camera_mover and it doesn't seem to effect the speed for ";" and ".". I just learned that you can left click on the stars and the game zooms to the star, maybe that is what the speed controls?

I am glad that the looking around with the mouse in the starview isn't as sensitive as it is in the original game.

comment:2 Changed 23 months ago by dafioram

Owner: set to dreammaster
Resolution: fixed
Status: newpending

PR982 resolved this. Looks like you might also be working on this...

comment:3 Changed 23 months ago by dreammaster

Status: pendingclosed

Yes, I had a hunch, which turned out to be correct. The original game did the scene drawing a much higher, frenzier rate, than the ScummVM implementation does. We have to support a variety of systems, so I didn't want to waste so many cycles on needless drawing and capped the frame rate to 30 frames per second. Unfortunately, as it turned out, a few places actually did rely on the high rate, and the star control is one of them, since it updated the camera during redrawing.

To get around this, I simply call to update the camera every 10ms in the main event loop. That gives the camera a chance to update more frequently without requiring any other changes to the star control logic. And I do want to keep the actual matrix calculations identical to the original.. I still need to eventually figure out why movements to center/lock on the 2nd and 3rd star take the wrong route, so I need the figures in both the original and ScummVM to be equal so I can I figure where it goes wrong.

comment:4 Changed 23 months ago by dreammaster

I'll have a look at your pull request now.

Note: See TracTickets for help on using tickets.