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

Certain files don't read properly #4

Closed
ghost opened this issue Nov 14, 2016 · 7 comments
Closed

Certain files don't read properly #4

ghost opened this issue Nov 14, 2016 · 7 comments

Comments

@ghost
Copy link

ghost commented Nov 14, 2016

Hi Buzz, thanks for putting this together! Everything works great for short clips (15 seconds, 3-5MB) but I've tried a couple longer ones (2 min, ~11MB) and they do not grab the data. I tried on your demo page as well and it's having the same issue. Any idea what might be causing this?

As a note, the desktop app for MediaInfo pulls the data correctly for both files.

The screenshot below shows the longer clip on top and shorter one below it.
image

@ghost ghost closed this as completed Nov 14, 2016
@ghost
Copy link
Author

ghost commented Nov 14, 2016

Solution was to increase chunk size. Misread the size originally.

@buzz
Copy link
Owner

buzz commented Nov 14, 2016

I successfully read files several GiB in size.

But I also witnessed the behaviour you are describing. I didn't invest time investigating the issue further as this is a proof-of-concept project.

@buzz
Copy link
Owner

buzz commented Feb 13, 2017

Re-opening as I don't see this entirely solved.

The result should be the same independent of the chunk size.

If somebody would like to investigate please go ahead!

@SaintFlipper
Copy link

Hey @buzz
I'm also not convinced that #6 which you just closed is the same thing. Those 2 files I put on Dropbox are only around 40MB, but I have other .mp4 and .mpg files that are well over 900MB which are handled just fine.
Also I don't see mediainfo.js hanging or throwing an exception on these files, it just doesn't extract any details.
Thanks

@buzz buzz changed the title Longer files don't read properly Certain files don't read properly Feb 13, 2017
@buzz
Copy link
Owner

buzz commented Feb 13, 2017

Hi @SaintFlipper
I have to admit I can't prove that this is actually the same bug. As you pointed out it seems this is not about file size. I changed the misleading title of this issue to reflect this.

But then all cases I found so far that show the same poor output, and work without problem with command-line mediainfo, are solved by increasing the chunk size.

This is why I'm fairly certain that this is the same thing.

I propose we track further investigation in this issue until we have proof your theory is right and this are actually two unrelated bugs.

@SaintFlipper
Copy link

Thanks @buzz
Fingers crossed that they are both caused by the same underlying issue, rather than trying to track down two separate things. Let me know if you'd like any more pass/fail test videos as I have many of various sizes.
Thanks for all the good work on this by the way. Much appreciated and very useful.

@buzz
Copy link
Owner

buzz commented Mar 19, 2020

Should be fixed in v0.0.3. Otherwise please re-open.

@buzz buzz closed this as completed Mar 19, 2020
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

2 participants