Fix failing reference tests 3341/13 and 3341/14 about ebur128

[l0calh05t]

I had a quick look into this, and I have a question: Where does your chunk size of 100 ms come from? I ask, because if I go down to a chunk size of 10 ms (20 ms also seem to work, anything larger causes the tests to fail again) like so:

        for chunk in samples.chunks(2 * 48_000 / 100) {
            e.add_frames_i32(chunk).expect("Failed to analyze samples");

            let loudness = e
                .loudness_momentary()
                .expect("Failed to get momentary loudness");
            if loudness > max_loudness {
                max_loudness = loudness;
            }
        }

The tests pass (the 8s in the second test have to be replaced by 80s obviously).

[l0calh05t]

Just for clarification, I looked at the chunk size because these tests ask for a maximum momentary loudness and while even momentary loudness has a 400ms window, it is still a sliding window and the maximum can occur at any point. loudness_momentary returns the current loudness at the end of a chunk, but the maximum could have been at any point between.

(I didn't check the i%8 expressions in the second test in too much detail, it may no longer represent exactly what the spec mentions)

[sdroege]

Makes sense. I didn't put much thought into the chunk size :) can you update the PR to use 10ms? That seems like a nicer number to understand the calculations around it.

Thanks!

[l0calh05t]

Makes sense. I didn't put much thought into the chunk size :) can you update the PR to use 10ms? That seems like a nicer number to understand the calculations around it.

Thanks!

Done 🙂