Opened 19 months ago

Last modified 18 months ago

#15683 new defect

SCI: SQ5: Two more rooms with speed problems

Reported by: GermanTribun Owned by:
Priority: normal Component: Engine: SCI
Version: Keywords:
Cc: Game: Space Quest 5

Description

System: Windows 10 Pro
ScummVM: 2.9.0
Game: Space Quest V
Version, English, German (Frensh perhaps?)

I tested this with the English and German SQ5 versions, to make sure.

After the speed problems in the into, two more rooms were identified by me, where background animations go way too fast in ScummVM, even after your speed throttling.

The first room is the hallway inside the Eureka. The lights on the ceiling flicker really fast, while in actuality the change in color was supposed to be way slower. The same goes for the red flickering at the right. The other animations also go too fast.

The other problematic area is the inside of WD40's ship The color cycling in all the red panels goes way too fast, destroying the intended effect.

I do also include the color cycling of the Sierra logo. While it isn't out of control fast, on authentic hardware it would not be this fast.

Considering how the game was made, the have to be more problematic rooms.

Attachments (4)

eureka.png (50.9 KB ) - added by eriktorbjorn 18 months ago.
sq5.005 (41.9 KB ) - added by eriktorbjorn 18 months ago.
wd40.png (39.3 KB ) - added by eriktorbjorn 18 months ago.
sq5.012 (32.9 KB ) - added by eriktorbjorn 18 months ago.

Download all attachments as: .zip

Change History (8)

by eriktorbjorn, 18 months ago

Attachment: eureka.png added

by eriktorbjorn, 18 months ago

Attachment: sq5.005 added

comment:1 by eriktorbjorn, 18 months ago

I assume this is the first of the rooms you mean:


This is not meant as the proper patch - I don't have the expertise for that, and I don't know what the speed is supposed to be - but just an experiment with the speed throttling...

--- a/engines/sci/engine/kmisc.cpp
+++ b/engines/sci/engine/kmisc.cpp
@@ -168,6 +168,10 @@ reg_t kGameIsRestarting(EngineState *s, int argc, reg_t *argv) {
                        }
                        break;
                }
+               case 225:
+                       s->_throttleTrigger = true;
+                       neededSleep = 90;
+                       break;
                default:
                        break;
                }

comment:2 by GermanTribun, 18 months ago

Yep, that's the room.

by eriktorbjorn, 18 months ago

Attachment: wd40.png added

by eriktorbjorn, 18 months ago

Attachment: sq5.012 added

comment:3 by eriktorbjorn, 18 months ago

And this (room 330) would be the second one?


Again, I'm not the right person to fix it. I've played through the game only once, and that was - as far as I remember - using DOSBox. Way before ScummVM supported the game, but also way after the hardware the game was designed for.

comment:4 by GermanTribun, 18 months ago

That's the room.

Note: See TracTickets for help on using tickets.