Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

firebeat: Use CHDs with proper subchannel data for Keyboardmania 1st/2nd #7873

Merged
merged 1 commit into from Mar 19, 2021

Conversation

987123879113
Copy link
Contributor

@987123879113 987123879113 commented Mar 19, 2021

This is related to the recent #7818.

The CHDs included in that PR didn't include the subchannel data so I got the original data from Jennifer Taylor and rebuilt them using bin/toc files extracted with cdrdao.

This fixes 2 second audio offset issues in Keyboardmania 2nd Mix.
There is still a 2 second audio offset issue in Keyboardmania 1st Mix/Keyboardheaven which I believe could possibly be due to an incomplete implementation of mixed mode LBA offsets (see: #7822).

All dumps are to be credited to Jennifer Taylor.


DISK_REGION( "ata:1:cdrom" ) // audio CD-ROM
DISK_IMAGE_READONLY( "gu974-ka a02", 1, SHA1(e66930f965b1aa1a681ab696302a04958dc8a334) ) // identical to jaa02 image
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The .img is identical but the subchannel data isn't.

> sha1sum -b GQ974-JAA02/GQ974-JA\ A02.img GQ974-JAA02/GQ974-JA\ A02.sub GU974-KAA02/GU974-KA\ A02.img GU974-KAA02/GU974-KA\ A02.sub 
cc7d90d902c0e5553988149bc57a05c3cac472b5 *GQ974-JAA02/GQ974-JA A02.img
7f3a2f6a19bdc01bee2e090e696b3c3c6b6b2056 *GQ974-JAA02/GQ974-JA A02.sub
cc7d90d902c0e5553988149bc57a05c3cac472b5 *GU974-KAA02/GU974-KA A02.img
ec9c52ebcd1a769748628a4ea6d334e93cbed9ae *GU974-KAA02/GU974-KA A02.sub

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

subchannel data tends to be less repeatable on multiple reads, so it's possible the discs are still identical, just you'd get slightly different subchannel data each time, it's one of the issues with CD dumping (and why in some cases, it isn't included unless it's needed for protections etc.)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That’s why we have things like cdparanoia for dealing with it. What tools were actually used to produce the dumps in the first place?

Copy link

@ghost ghost Mar 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

afaik for subchannel data even tools like cdparanoia don't really help, there are long and heated debates on the subject to the point people have literally just given up, because it became a blocking issue.

@cuavas cuavas merged commit f15e301 into mamedev:master Mar 19, 2021
@987123879113 987123879113 deleted the firebeat_update_kbm branch September 18, 2021 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants