Opened 10 years ago

Last modified 3 years ago

#6730 new defect

SCUMM/HE: Putt-Putt Goes to the Moon MIDI Music identified as MT-32, rather than GM

Reported by: SF/nukeykt Owned by:
Priority: normal Component: Engine: SCUMM
Version: Keywords:
Cc: Game: Putt-Putt Goes to the Moon

Description

DOS Putt-Putt Goes to the Moon's musics are GM. But ScummVM considers they are MT32.

Ticket imported from: bugs/6730.

Attachments (12)

scummvm00004.jpg (29.8 KB ) - added by SF/nukeykt 10 years ago.
scummvm00001.jpg (32.7 KB ) - added by SF/nukeykt 10 years ago.
scummvm00002.jpg (30.9 KB ) - added by SF/nukeykt 10 years ago.
scummvm00003.jpg (35.1 KB ) - added by SF/nukeykt 10 years ago.
scummvm00005.jpg (30.3 KB ) - added by SF/nukeykt 10 years ago.
scummvm00006.jpg (29.6 KB ) - added by SF/nukeykt 10 years ago.
debug.txt (6.5 KB ) - added by SF/nukeykt 10 years ago.
vc.png (13.2 KB ) - added by SF/nukeykt 10 years ago.
ppm.diff (563 bytes ) - added by SF/nukeykt 10 years ago.
ppm2.diff (3.5 KB ) - added by SF/nukeykt 10 years ago.
PUTTMOON.DRV (185.2 KB ) - added by SF/nukeykt 10 years ago.
PUTTMOON.2.DRV (203.6 KB ) - added by SF/nukeykt 10 years ago.

Download all attachments as: .zip

Change History (28)

comment:1 by digitall, 10 years ago

Summary: Putt-Putt Goes to the Moon's musicSCUMM: Putt-Putt Goes to the Moon MIDI Music identified as MT-32, rather than GM

comment:2 by digitall, 10 years ago

nukeykt: Thank you for your bug report. However, you have omitted several details which will be required in order to investigate this bug. 1. Which operating system are you using? 2. Which version of ScummVM are you using? (This is shown on the launcher screen below the logo) 3. Please attach a file listing of your Putt-Putt goes to the Moon datafiles to this bug as a text file attachment, preferably with file md5sums? The output of a tool such as http://md5summer.org/ is optimal. 4. Please attach screenshots of your Options->Audio, Options->MIDI and Options->MT32 setting screens to this bug as JPEG or PNG. (Attachments are limited to below 256K per item, so the bitmap output from Alt-S (screenshot) will need to be converted with a tool such as GIMP or imagemagick to reduce size).

Please can you ensure that you provide the information in points 1 to 3 initially on any future bug reports you file. Thanks.

comment:3 by digitall, 10 years ago

There are a few reasons why this can occur. It could be variant datafiles, or an incorrect setting in the Audio/MIDI/MT32 screens, or due to the SCUMM engine music driver autodetection getting the wrong music driver, but replicating this is tricky as it depends on the MIDI / music drivers provided to ScummVM from the OS, so we need a bit more information to track this down.

by SF/nukeykt, 10 years ago

Attachment: scummvm00004.jpg added

comment:4 by SF/nukeykt, 10 years ago

  1. Windows 8
  2. ScummVM 1.8.0git(commit 6f343afe8b5afbdb4690adcc95b00760dad4a52d)
  3. PUTTMOON.HE0 780e4a0ae2ff17dc296f4a79543b44f8 PUTTMOON.HE1 f02f36270d684e0c245d0fd0519b5e7b PUTTMOON.TLK bfecc91f0bf7b86acd7f82a3dfb53ca5

comment:5 by digitall, 10 years ago

What about the MIDI tab for Options and Edit-Game (Override?)?

I would suggest that you run scummvm from a command window with the line "scummvm -d 5" and see if any messages relating to MIDI or sound output are produced.

by SF/nukeykt, 10 years ago

Attachment: scummvm00005.jpg added

comment:6 by SF/nukeykt, 10 years ago

console output and screenshots

comment:7 by wjp, 10 years ago

What exactly is making you conclude that "ScummVM considers they are MT32"?

by SF/nukeykt, 10 years ago

Attachment: vc.png added

comment:8 by SF/nukeykt, 10 years ago

vc 2013 debugging(engines\scumm\imuse\instrument.cpp:420)

by SF/nukeykt, 10 years ago

Attachment: ppm.diff added

comment:9 by SF/nukeykt, 10 years ago

I made small fixing patch

by SF/nukeykt, 10 years ago

Attachment: ppm2.diff added

comment:10 by SF/nukeykt, 10 years ago

Updated patch(Putt-Putt Goes to the Moon Demo uses Putt-Putt Joins the Parade music)

comment:11 by digitall, 10 years ago

Thank you for the patches. They will be reviewed by the SCUMM engine developers when they have time and then merged.

comment:12 by Kirben, 10 years ago

Why exactly do you think the music was designed for General MIDI?

Unfortunately the setupcd program included with the original DOS version doesn't offer any choice for General MIDI or Roland MT32, so I can't compare under DOSBox

comment:13 by digitall, 10 years ago

@kirben: I don't have this game, so can't give direct evidence, but based on http://www.mobygames.com/game/win3x/putt-putt-goes-to-the-moon/reviews/reviewerId,246035/, the Windows version at least was intended for use with Roland GS.

The relevant section is:

comment:14 by Kirben, 10 years ago

All the Windows ports uses digital music, although that confirms the digital music was recorded from a Roland GS for this game.

by SF/nukeykt, 10 years ago

Attachment: PUTTMOON.DRV added

comment:15 by SF/nukeykt, 10 years ago

i have modified PUTTMOON.DRV to MIDI output on SB setting

comment:16 by Kirben, 10 years ago

That PUTTMOON.DRV file, doesn't seem to work with the full DOS version of the game. Which MidiSoundDriver is expected by your changes?

comment:17 by SF/nukeykt, 10 years ago

sbfm.adv

by SF/nukeykt, 10 years ago

Attachment: PUTTMOON.2.DRV added

comment:18 by SF/nukeykt, 10 years ago

I created new modification of PUTTMOON.DRV set MidiSoundDriver=genmidi.adv

comment:19 by Kirben, 10 years ago

Unfortunately both versions of your PUTTMOON.DRV file, cause the game to lock up (not startup), when I tried with to use with suggested sound settings use under the DOXBox emulator.

comment:20 by SF/nukeykt, 9 years ago

I've extracted XMI files, then converted it to MID, and played back through Roland SC-55mkII in GM mode. Results almost exacts to Windows version. See: youtube.

comment:21 by sev-, 3 years ago

Summary: SCUMM: Putt-Putt Goes to the Moon MIDI Music identified as MT-32, rather than GMSCUMM/HE: Putt-Putt Goes to the Moon MIDI Music identified as MT-32, rather than GM
Note: See TracTickets for help on using tickets.