search

Ymagis / ClairMeta

Clairmeta is a python package for Digital Cinema Package (DCP) probing and checking.
BSD 3-Clause "New" or "Revised" License
84 stars 22 forks source link

DCP Naming Convention checks #235

Closed bquandt closed 10 months ago

bquandt commented 1 year ago

I think its wrong, potentially misleading, to use the word 'warning' in DCP naming convention checks.

While I think it would be great for the ISDCF recommendations to have been made required within the XML's itself, that is not what happened. Usage of the English word 'warning' is too closely tied to being in error or being wrong.

I suggest a better classification of such checks, e.g. maybe 'recommendation' vrs 'warning' would be better?

I think the word 'warning' creates confusion for a person thinking a DCP is 'bad' (aka a false positive/negative) when it really is not.

It would be nice to change language to be in alignment with what ISDCF/SMPTE seemed to do, use the word 'recommendation.'

remia commented 1 year ago

Sorry for the silence @bquandt, I agree with your point. Maybe we should turn the DCNC related warnings into info, which is a level of errors below warning in ClairMeta default reporting.

kieranjol commented 1 year ago

Yeah, in the hundreds of dcps that were qc’d in an archive I worked for, I think maybe one ever followed the convention perfectly. And he worked in the archive :)

worldpoop commented 1 year ago

"Alert"? "Advisory"? There may be a place for "Warning" sometimes, but most seem just to be advisories rather than notices of real potential problems. It's a bit of a mouthful, but "Advisory" is pretty accurate.

On Sat, Oct 14, 2023 at 10:05 AM kieranjol @.***> wrote:

Yeah, in the hundreds of dcps that were qc’d in an archive I worked for, I think maybe one ever followed the convention perfectly. And he worked in the archive :)

— Reply to this email directly, view it on GitHub https://github.com/Ymagis/ClairMeta/issues/235#issuecomment-1763052689, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNKDUAPB3IMZLHUJSY5QXDX7LA7JANCNFSM6AAAAAA4N6RRM4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>