MediaConch_SourceCode icon indicating copy to clipboard operation
MediaConch_SourceCode copied to clipboard

verbosity attribute in MediaTrace

Open dericed opened this issue 9 years ago • 8 comments

Some implementation checks require --ParseSpeed 1 or other verbose MediaTrace options, such as testing to make sure all Seek elements resolve (this fails on low verbosity mediatrace, since not all Clusters are noted).

I propose a verbose attribute to be added where ever the parser attribute is used, so in block, ref, and MediaTrace.

dericed avatar May 29 '16 15:05 dericed

I don't like we terms ParseSpeed(no real meaning) and verbosity(it is not about verbosity but how much data we check).

completeness="full", completeness="fast", completeness="medium"? (I don't like the idea of having numbers, is "1" full or fast and is "9" full or fast?)

JeromeMartinez avatar May 29 '16 16:05 JeromeMartinez

you're right, i remember we debate this before. I was on the side of coverage

dericed avatar May 29 '16 16:05 dericed

ho, I forgot the debate.

coverage="full" (--ParseSpeed=1), coverage="fast" (--ParseSpeed=0), coverage="medium" (--ParseSpeed=0.5 and default)?

JeromeMartinez avatar May 29 '16 16:05 JeromeMartinez

I suggest number with coverage with words as aliases to numbers.

dericed avatar May 29 '16 19:05 dericed

which number? which alias?

JeromeMartinez avatar May 29 '16 19:05 JeromeMartinez

I prefer something unique (number or text), having 2 methods for saying the same thing may be confusing.

JeromeMartinez avatar May 29 '16 19:05 JeromeMartinez

Check the man page for the flac utility. There is something like this in the compression option.

dericed avatar May 29 '16 20:05 dericed

we talk of an XML output, so a parser will have to support both if we accept both. This is not similar to command line options

JeromeMartinez avatar May 29 '16 20:05 JeromeMartinez