Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#9741 closed defect (fixed)

SCI: Shivers: Intro video too small

Reported by: legluondunet Owned by: csnover
Priority: normal Component: Engine: SCI
Keywords: sci32 Cc:
Game: Shivers 1

Description

Hello,

when I launch Shivers, the intro video played is too small:

on scummvm last git from 29 april 2017:
https://goo.gl/photos/5TCuiVFB8t23wPUM6

on Dosbox:
https://goo.gl/photos/NCCnN34udyQW6ftB7

Thank you for your help.

Change History (7)

comment:1 by Kurufinwe21, 2 years ago

Component: --Unset--Engine: SCI
Game: Shivers 1

It works fine for me, as long as the large videos are enabled in the engine tab of the launcher. (Maybe that should be the default?)

comment:2 by csnover, 2 years ago

Summary: Shivers Intro video too smallSCI: Shivers: Intro video too small

Thanks for your report!

SCI system scripts run a benchmark at startup of the game engine to detect the video performance of the system and adjust visual effects accordingly. This benchmarking code does not always accurately detect the performance of modern systems and can cause the game to think that the video card is too slow to be able to render video at full size.

I will look more closely at how benchmarking is being handled to see if there is any reason to not simply disable it by ignoring writes to the backend during benchmarking. In the meantime, once you start the game, you can go to the in-game settings and toggle it from “Halfscreen” to “Fullscreen”, which will restore rendering of the full sized videos.

comment:3 by csnover, 2 years ago

Keywords: sci32 added

comment:4 by legluondunet, 2 years ago

In he original game, they are using scanlines to play the video in a bigger size.
I try different options in the graphic tab, the video always play in the original size, it is not stretched.

comment:5 by csnover, 2 years ago

I try different options in the graphic tab, the video always play in the original size, it is not stretched.

The setting for video playback size (“Halfscreen”/“Fullscreen”) is in the game itself, not in ScummVM. Go to the in-game menu.


Investigating this issue, I’ve determined that the SCI game script benchmarking code is never going to work correctly and needs to be disabled. (Calling to OSystem::updateScreen on every frameout during benchmarking makes the system appear slower than it is; not calling it causes the benchmarking counter to overflow because it runs too fast.)

comment:6 by csnover, 2 years ago

Owner: set to csnover
Resolution: fixed
Status: newclosed

Game script video benchmarking has been disabled across all SCI32 games in commit 8d94a046052bc5c041831a77b2a9a8b59e1f527e, so Shivers should now always default to “fullscreen” video mode when running ScummVM build 1.10.0git-3236 or later.

Note that any saved games already created in “halfscreen” mode will need to be adjusted by going to the in-game settings menu and changing “halfscreen” to “fullscreen”.

comment:7 by legluondunet, 2 years ago

Very good job, thank you!

Note: See TracTickets for help on using tickets.