[Problem] Error reporting BD-R disc type #466

New Issue

claunia · 2026-01-29T16:16:55Z

claunia commented

2026-01-29 16:16:55 +00:00

Originally created by @FoxhackDN on GitHub (Oct 20, 2022).

Originally assigned to: @mnadareski on GitHub.

Version
What version are you using?

[*] WIP release (2.3-949 / 2.3-952)

Build
What runtime version are you using?

[*] .NET Framework 4.8 running on (Operating System)

Describe the issue
I burned a BD-R to test MPF's work in progress disc type detection and display in the submissioninfo file. This failed with this BD-R. The _disc.txt file output by DiscImageCheck correctly displayed "DiscTypeIdentifier: BDR", but the submission info said "Reported Disc Type: ecific: 00".

This bug was fixed in 2.3-952, but now MPF reports the following: "Reported Disc Type: 00, BDR, 00, BDR, 00, BDR, 00, BDR"

The reason is that the _disc.txt file as generated by DiscImageCreator has the disc information ... four times. Even if it's a single layer disc.

To Reproduce
Steps to reproduce the behavior:

Dump a BD-R
Get wrong info in submissioninfo.txt due to badly formatted _disc.txt file

Expected behavior
Disc Type should say 00, BDR once.

Originally created by @FoxhackDN on GitHub (Oct 20, 2022). Originally assigned to: @mnadareski on GitHub. **Version** What version are you using? - [*] WIP release (2.3-949 / 2.3-952) **Build** What runtime version are you using? - [*] .NET Framework 4.8 running on (Operating System) **Describe the issue** I burned a BD-R to test MPF's work in progress disc type detection and display in the submissioninfo file. This failed with this BD-R. The _disc.txt file output by DiscImageCheck correctly displayed "DiscTypeIdentifier: BDR", but the submission info said "Reported Disc Type: ecific: 00". This bug was fixed in 2.3-952, but now MPF reports the following: "Reported Disc Type: 00, BDR, 00, BDR, 00, BDR, 00, BDR" The reason is that the _disc.txt file as generated by DiscImageCreator has the disc information ... four times. Even if it's a single layer disc. **To Reproduce** Steps to reproduce the behavior: 1. Dump a BD-R 2. Get wrong info in submissioninfo.txt due to badly formatted _disc.txt file **Expected behavior** Disc Type should say 00, BDR once.

claunia added the bug label 2026-01-29 16:16:55 +00:00

claunia closed this issue

2026-01-29 16:16:55 +00:00

claunia commented

2026-01-29 16:16:56 +00:00

@mnadareski commented on GitHub (Oct 26, 2022):

Mentioned offline but this won't be fixed for next stable. Possibly in the future (or generated directly from the PIC. Note for self: Look at DIC code to see the structure of the data that holds the layerbreak in the PIC and make it into a proper parsed out object.

@mnadareski commented on GitHub (Oct 26, 2022): Mentioned offline but this won't be fixed for next stable. Possibly in the future (or generated directly from the PIC. Note for self: Look at DIC code to see the structure of the data that holds the layerbreak in the PIC and make it into a proper parsed out object.

claunia commented

2026-01-29 16:16:56 +00:00

@mnadareski commented on GitHub (Jul 6, 2023):

Try this out when it's built: 7bfe174680

@mnadareski commented on GitHub (Jul 6, 2023): Try this out when it's built: https://github.com/SabreTools/MPF/commit/7bfe174680d381baad05e4606e489351e18f23c8

claunia commented

2026-01-29 16:16:56 +00:00

@mnadareski commented on GitHub (Jul 7, 2023):

Fully fixed by 4d520d7d63

@mnadareski commented on GitHub (Jul 7, 2023): Fully fixed by https://github.com/SabreTools/MPF/commit/4d520d7d6317e2a8ccfbdde7c1640834fd9f7227

claunia referenced this issue

2026-01-29 16:25:15 +00:00

[PR #635] Add a GUI for MPF.Check #1077

Sign in to join this conversation.