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

DCDM - more checks needed #140

Open
kieranjol opened this issue Mar 10, 2020 · 4 comments
Open

DCDM - more checks needed #140

kieranjol opened this issue Mar 10, 2020 · 4 comments

Comments

@kieranjol
Copy link
Contributor

We have a sequence where one corrupted tif resulted in DCP-O-Matic not being able to make a DCP. After doing an audit in exiftool, we found a corrupt frame. ClairMeta approved this sequence as the filesize, mimetype and sequence continuity were all ok. I suggest some further probing, for example:
Width/Height, Bit Depth , some sort of check that a valid tiff header is present. Whatever is the least intensive for the greatest chance of detecting corruption.
Here's the exiftool report on the corrupt frame:

exiftool E:\DCDM\video\Reel_3\DCDM_Tiff16bit-XYZ_DCI2K-Scope_Reel03_075580.tif
ExifTool Version Number         : 11.19
File Name                       : _DCDM_Tiff16bit-XYZ_DCI2K-Scope_Reel03_075580.tif
Directory                       : E:/DCDM/video/Reel_3
File Size                       : 10 MB
File Modification Date/Time     : 2019:02:08 14:41:08+00:00
File Access Date/Time           : 2019:02:08 14:41:08+00:00
File Creation Date/Time         : 2019:02:08 14:41:08+00:00
File Permissions                : rw-rw-rw-
File Type                       : TIFF
File Type Extension             : tif
MIME Type                       : image/tiff
Exif Byte Order                 : Little-endian (Intel, II)

and here's the previous frame which is fine:

ExifTool Version Number         : 11.19
File Name                       : DCDM_Tiff16bit-XYZ_DCI2K-Scope_Reel03_075579.tif
Directory                       : New folder
File Size                       : 10 MB
File Modification Date/Time     : 2018:05:11 02:23:31+01:00
File Access Date/Time           : 2020:03:10 09:44:41+00:00
File Creation Date/Time         : 2020:03:10 09:44:41+00:00
File Permissions                : rw-rw-rw-
File Type                       : TIFF
File Type Extension             : tif
MIME Type                       : image/tiff
Exif Byte Order                 : Little-endian (Intel, II)
Image Width                     : 2048
Image Height                    : 858
Bits Per Sample                 : 16 16 16
Compression                     : Uncompressed
Photometric Interpretation      : RGB
Strip Offsets                   : 8
Samples Per Pixel               : 3
Rows Per Strip                  : 858
Strip Byte Counts               : 10543104
Planar Configuration            : Chunky
Image Size                      : 2048x858
Megapixels                      : 1.8

Best,

Kieran.

@remia
Copy link
Collaborator

remia commented Mar 10, 2020

Hi @kieranjol ,

Just to be sure, what version of ClairMeta where you using, and was it the DSM or DCDM checker ?

In the latest versions, we no longer use Mime type as it was found to be unreliable, we check for:

That being said we could add more and also make sure there is no change in the way mediainfo extract those that would make ClairMeta outdated. Would be nice to have the corrupted frame and another valid one (no matter which). Will probably look into this next week if there is a need.

Thanks

@kieranjol
Copy link
Contributor Author

kieranjol commented Mar 10, 2020

Hi, thanks for getting back so quick and I apologise for having so many things missing from my report.
We were using the DCDM checker.
python -m clairmeta.cli check -type dcdm "C:\Users\blaaa\New folder"

and i'd say i't sht latest version cos we reinstalled lately and pip install -U clairmeta says it's the ;latest version:

C:\Users\blaaa>pip show clairmeta
Name: clairmeta
Version: 1.0.5
Summary: Digital Cinema Package (DCP) probing and checking utility
Home-page: https://github.com/Ymagis/ClairMeta
Author: Ymagis
Author-email: support.sol@ymagis.com
License: BSD
Location: c:\users\blaaa\appdata\roaming\python\python37\site-packages
Requires: lxml, pyopenssl, shutilwhich, dicttoxml, python-dateutil, xmltodict, pycountry, six
Required-by:

I think the mediainfo checks are not running on my end - is there a chance that this is only in git-master but not in the latest pip version?

I'll shoot you an email now too.

Thanks!

@remia
Copy link
Collaborator

remia commented Mar 10, 2020

It should be deployed!

Maybe mediainfo was not found on the host, but you should see warning in that case. Or there is a bug somewhere / version incompatibility, will have to look at the files when you send them and let you know.

@kieranjol
Copy link
Contributor Author

kieranjol commented Mar 10, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants