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

BDInfo does not create report #35

Open
HolyGit12 opened this issue Jan 9, 2023 · 17 comments
Open

BDInfo does not create report #35

HolyGit12 opened this issue Jan 9, 2023 · 17 comments

Comments

@HolyGit12
Copy link

BDInfo does not create a report (does not save and does not show it) when the disc label contains a characters which cannot be written.
This problem appears of course only when I open ISO-images.

Example:
Create an ISO with ImgBurn and add as Volume Label (UDF) 'Test: 1'

The disc label contains a colon and cannot be used as filename.
BDInfo will read the ISO but won't create a report.

You have to replace such characters by '-' or ' ' or eliminate them.

@Hellboy00000
Copy link

I created an ISO with tsMuxwer, add a colon in disk label and work ok. What is the duration of your test file?

@HolyGit12
Copy link
Author

I tested with UHD. It was 60 GB I guess.

@HolyGit12
Copy link
Author

HolyGit12 commented Jan 16, 2023

I also tried to generate the iso with tsMuxwer. But also for that iso no report is created.

@Hellboy00000
Copy link

  1. Try to extract the ISO and see if the folder work.
  2. Share a sample. You can use the tool that "UniqProject" created "Bitrate is 0 if is 3D Bluray with 50Hz content #24"
    to created the sample. Or share the "Test: 1" you created.
  3. Try this. Uncheck in BDInfo settings -> Filter playlists with length (and then try the other Filter playlist)

@HolyGit12
Copy link
Author

Not sure whether we understand each other correctly.
The iso works.
BDInfo is scanning the files.
I only cannot save a report. Not automatically and also not manually since the report form does not pop up.

I created two iso for testing. The 1st one is small. The second one the whole film
https://www.dropbox.com/s/pke6tsisspsyuf9/Test%20-%202%20%28tsMuxer%29.iso?dl=0
https://www.dropbox.com/s/soieut9agomm6y0/Der%20Geist%20im%20Glas.iso?dl=0

They have the same problem with bitrate of 0 - fitting to your link.
The second one has also an audio stream with a bitrate of 0
But this was not my original problem

@Hellboy00000
Copy link

Ok. Yes the problem is the invalid characters. I found other 2 bug. The first is, BDInfo contain an exception code for this but never enter and the second is an invalid cross tread operation that happened few times.

I fixed the exception and the invalid characters. I upload a fixed version letter.

@UniqProject
Copy link
Owner

UniqProject commented Jan 20, 2023

Hi guys,

just ran few tests with both ISO images.
Reports work for both images, although for the full movie it must first read through the full movie to finish the basic detection.
Bitrate detection works only for Dolby Digital stream though.

Hower, the files seem not to be fully BluRay compliant, because the headers for both AVC and MPEG2 streams are not detected as such in the MPEG Transport stream, and therefore the stream parsing code is not executed at all.

The stream infos listed in the list are only read from the playlists / clipinfo files.

Ran a reencode with handbrake and remuxed with TSmuxeR, both images are detected without any problem, regardless of the UDF label.
However, reencoding just the video stream and remux using original audio streams from the full movie results in a similar problem. AVC and Dolby Digital streams detected, MPEG2 not.

Image 1 Initial Scan
Image 1 Full Scan
Image 1 Reencoded
Image 2 Initial Scan
Image 2 Full Scan
Image 2 Reencoded

The MPEG TS parsing was written by the original author back in 2011 and only modified to support ISO images by me. It still may contain bugs, however none of commercially mastered BluRay discs I came across until now had this problem.

Will have a closer look at the code, but it has a very low priority for now.

If someone wants to submit a patch, you are very welcome to do so.

@UniqProject
Copy link
Owner

Ok, there were a few things wrong with those images, that I actually missed the report autosaving problem.

However, I have it fixed with a571337

@UniqProject
Copy link
Owner

Also, with changes in b5cd753 and 74b9e2f scanning is much faster for those images.

However, MPEG2 Audio streams are still not parsed, and as result there is also no bitrate detection for them

@Hellboy00000
Copy link

Hellboy00000 commented Jan 21, 2023

Thanks UniqProject. I fixed the label with this but your method is better because only remove invalid characters.

public static string RemoveSpecialCharacters(string str)
{
string s = string.Empty;
foreach (char c in str)
{
if (char.IsLetterOrDigit(c) || char.IsWhiteSpace(c) || c == '.' || c == '_' || c == '-')
{
s += c;
}

            if (string.IsNullOrWhiteSpace(sb))
            {
                s = "No Label or Invalid Label";
            }
        }
        return s.ToString();
    }

string reportName = string.Format( CultureInfo.InvariantCulture,
"BDINFO.{0}.txt",
FormMain.RemoveSpecialCharacters(BDROM.VolumeLabel));

You can try to change: reportName = ToolBox.GetSafeFileName(reportName); for
ToolBox.GetSafeFileName(BDROM.VolumeLabel));

@Hellboy00000
Copy link

I change this "Exception" in FormMain.cs because never used.

private void GenerateReportCompleted(object sender, RunWorkerCompletedEventArgs e)
{
HideNotification();
if (e.Result != null)
{
if (e.Result.GetType().Name == "FormReport")
{
((Form)e.Result).Show();
}
else if (((Exception)e.Result).Message != null) //e.Result.GetType().Name == "Exception" - changed this because never enter
{
string msg = string.Format(
"{0}", ((Exception)e.Result).Message);

                MessageBox.Show(msg, "BDInfo Error",
                    MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
        }
        buttonViewReport.Enabled = true;
    }

@UniqProject
Copy link
Owner

Please try the latest beta release v0.7.6.0b and see if problem is fixed.

@HolyGit12
Copy link
Author

HolyGit12 commented Jan 22, 2023

It seems to work and scanning is 15-30% faster (compared to 0.7.5.8b).
Thanks!

But:
Both versions show for some m2ts files of an image different sizes.
If this is the case 0.7.6 reports 1 packet more.

While comparing the reports I saw that there were also some correct changes for chapters.
But for some M2TS the chapters show a value for
Avg Frame Size
and for some these values are 0 bytes (for all chapters). Is this a bug or has it a reason?
The ones who show values seem to be the ones with the above mentioned changes in packet size.

@UniqProject
Copy link
Owner

Both versions show for some m2ts files of an image different sizes.
If this is the case 0.7.6 reports 1 packet more.

I haven't come across such a disk so far, so unfortunately can't tell what the problem is.

As for the rest, can you please test with the latest beta release v0.7.6.1b and see if this has fixed anything?

@HolyGit12
Copy link
Author

No problems so far. Looks good.

@HolyGit12
Copy link
Author

One small thing - can you format the bitrate of the video? You do it already for the audio and subtitles:

`Codec Bitrate Description


MPEG-H HEVC Video 30,311 kbps 2160p / 23.976 fps / 16:9 / Main 10 @ Level 5.1 @ High / 10 bits / HDR10 / BT.2020

  • MPEG-H HEVC Video 3,338 kbps 1080p / 23.976 fps / 16:9 / Main 10 @ Level 5.1 @ High / 10 bits / Dolby Vision / BT.2020

AUDIO:

Codec Language Bitrate Description


DTS:X Master Audio English 6679 kbps 7.1 / 48 kHz / 6679 kbps / 24-bit (DTS Core: 5.1 / 48 kHz / 1509 kbps / 24-bit)
DTS Audio English 256 kbps 2.0 / 48 kHz / 256 kbps / 24-bit`

The kbps for audio is in one columnn - but not the one for the video
(format ##.### kbps with a leading space)

@HolyGit12
Copy link
Author

Not sure whether I was clear what I meant
Can you please change

VIDEO:

Codec                   Bitrate             Description     
---------------         -------------       -----------     
MPEG-H HEVC Video       52,299 kbps         2160p / ***
* MPEG-H HEVC Video     69 kbps             1080p / ***

to

Codec                   Bitrate             Description     
---------------         -------------       -----------     
MPEG-H HEVC Video       52,299 kbps         2160p / ***
* MPEG-H HEVC Video         69 kbps         1080p / ***

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

No branches or pull requests

3 participants