Fraunhofer-IIS / mpeghdec

The Fraunhofer MPEG-H decoder (mpeghdec) is a C/C++ implementation of the MPEG-H Audio standard as defined in ISO/IEC 23008-3:2022
Other
61 stars 7 forks source link

mpeghUiManager outputs possibly wrongly formatted AudioSceneConfig XML #10

Closed BunnyMerz closed 2 months ago

BunnyMerz commented 2 months ago

After using the mpeghUiManager line command to output the AudioSceneConfig XML of the test given by test content repository, I got a weirdly formmated xml. Here is the output, and I manually eddited the content to hide what is inside AudioSceneConfig with ellipsis.

[0]
<?xml version="1.0" encoding="utf-8" ?>
<AudioSceneConfig uuid="E97E0000-0000-0000-0000-00002107BD59" version="11.0" configChanged="true"/>
<?xml version="1.0" encoding="utf-8" ?>
<AudioSceneConfig uuid="E97E0000-0000-0000-0000-00002107BD59" version="11.0" configChanged="false">
    ...
</AudioSceneConfig>

I'm not sure if the existance of the two roots elements are correct, nor what does the [0] at the start of the file mean. I tried this in multiple files, and got the same results.

mhildenbrand-iis commented 2 months ago

The "-xmlSceneState " does not write valid XML syntax to file but it is a text file containing a concatenation of XML snippets corresponding to a sequence of XML snippets returned by mpegh_UI_GetXmlSceneState() throughout the duration of the stream. "[]" denotes the frame number at which the XML snippet was received. In general you could have a stream where the configuration changes mid-stream for instance after 1000 frames. Then you get, in addition to what you have cited above, another such text block starting with "[1000]" and a different number for uuid all written to the same . I agree, this could be described clearer in our documentation.

BunnyMerz commented 2 months ago

That makes sense! Thanks