Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#9989 closed defect (invalid)

MOHAWK: Riven: Ambient sounds don't play after wood trolley reaches Jungle Island.

Reported by: macca8 Owned by: macca8
Priority: normal Component: Engine: Mohawk
Version: Keywords:
Cc: Game: Riven

Description

This issue relates to returning to Jungle Island in the wood trolley, specifically the period after it comes to rest (and the sound effects have completed) and before the player exits the trolley.

This bug was previously fixed (indirectly) back in August 2016, due to changes to handling ambient sounds made (by bgK) at that time.

As soon as the trolley came to rest, ambient sound would play (both before & after the cursor appeared)… until the recent overhaul of the Riven engine.

Since then, this behaviour has reverted back to its previous state (no sound). The player must now exit the trolley to trigger the ambient sound.

In the original CD version, there's no sound during this period, but note that an exchange of CDs also occurs here (the cursor appears immediately the next CD is loaded).

I'm hopeful that since this was fixed only recently, it can still be restored.

Current Daily Build: 1.10.0git4054-g96a1b70 (Jul 23 2017)
Last known with sound: 1.10.0git3512-gdeab63f (21 June 2017)
Game Version: English, 5-CD (contains v1.02 patch files)
Platform: Intel Mac (OS X 10.6.8)

The sound issue is the priority, but there's a secondary issue at play here (which was fixed long ago in ScummVM, but then broken sometime during 2015), and that's the delay before the appearance of the cursor.

Currently it stands at 8 seconds, whereas ScummVM had previously reduced this to 3 seconds (presumably eliminating the time taken to eject the CD). That was a very old fix and I appreciate it may no longer be attainable, but thought I'd mention it on the off chance it was still fixable.

Attachments (1)

riven-008.rvn (14.8 KB ) - added by macca8 7 years ago.
Pull lever in trolley

Download all attachments as: .zip

Change History (6)

by macca8, 7 years ago

Attachment: riven-008.rvn added

Pull lever in trolley

comment:1 by dafioram, 7 years ago

Tester OS: Win7-64
Delay = approx time between cart arriving and hand cursor appears
Ambient Sound = Ambient Sound plays while in cart before clicking out of trolley

Note, this requires that you take the trolley down and back up since if you just get in the trolley the ambient sound will play just fine for all versions.

Riven DVD on XP
Delay: 2.5 secs
Ambient Sound?: No
https://streamable.com/vdpph

Riven CD on XP
Delay: 2 secs
Ambient Sound?: No
https://streamable.com/dtz88

Riven DVD on ScummVM 96a1b70
Delay: 2 secs
Ambient Sound?: No
https://streamable.com/r6r4u

Riven CD on ScummVM 96a1b70
Delay: 4.5 secs
Ambient Sound?: No
https://streamable.com/jbdmh

Last edited 7 years ago by dafioram (previous) (diff)

comment:2 by dafioram, 7 years ago

I made a mistake... I had scummvm running riven in the background while I was playing Riven CD on XP. When I did it again I get no ambient sound for the Riven CD on XP. So none of them have ambient sound. I have upload a new video for the Riven CD on XP and that is now listed correctly above.

macca8 can you confirm that you are observing this when you play in the original engine?

comment:3 by macca8, 7 years ago

To clarify:

  • the original is installed on Mac OS 9.2.
  • it's NOT a full installation, so requires CD swapping.
  • the CDs are swapped before the cursor appears.
  • there is NO ambient sound (as stated above) until the player exits the trolley.

To confirm:

  • ambient sound did play in ScummVM (as described above) from Daily Build 1.9.0git-g4c7985c (17 Aug 2016) until the major engine overhaul.

Interested in seeing the delay time you've quoted for Riven CD on ScummVM is only 4.5 secs.
I've rechecked my version on ScummVM (I only use standard daily builds from ScummVM's buildbot) and there's a clear 8 secs delay.

comment:4 by macca8, 7 years ago

Owner: set to macca8
Resolution: invalid
Status: newclosed

Thanks to dafioram's tests, it's clear that no sound is present in the originals.
While ambient sound did play previously in ScummVM, closer inspection suggests this was the result of a conveniently placed glitch (now fixed), rather than a genuine fix.
I'm declaring this report invalid and closing the issue.

Regarding the cursor delay, while it's also clear that an extended delay exists in ScummVM for the CD version, inconsistencies between these versions make me doubt that it's worthwhile pursuing.

Nevertheless, if anybody wishes to do so, the source of this delay extension (at least in my case) lies somewhere inside the block of commits following Daily Build 1.8.0git2913-gd582c84 (23 March 2015) and ending no later than 4 May 2015 (or 1.8.0git2953?).

comment:5 by macca8, 7 years ago

I've opened the cursor delay issue in its own bug report (#10044).
Please post any related comments there. Thanks.

Note: See TracTickets for help on using tickets.