cta-wave / mezzanine

This repo contains scripts that will build annotated test content from specific source content, compatible with the WAVE device playback test suite.
BSD 3-Clause "New" or "Revised" License
2 stars 2 forks source link

fault in mezzanine QR code #50

Closed yanj-github closed 1 year ago

yanj-github commented 1 year ago

The last mezzanine QR code reports 00:60:00 instead of 01:00:00. I remember this issue has been corrected before but it seems the issue is back again. OF won't take 00:60:00 as a valid time data.

nicholas-fr commented 1 year ago

Thanks for raising this @yanj-github , I believe only the 30fps streams have this issue, could you confirm that's also what you see? I have a fix and will upload corrected mezzanine streams this week.

yanj-github commented 1 year ago

@nicholas-fr I will have a look, can I have the link to the latest mezzanine please?

nicholas-fr commented 1 year ago

Thanks @yanj-github, I thought you noticed the issue when using the test content, so I was wondering in which content you'd seen it. I have a fix for the issue and will create a pull request for the script. I uploaded new 30fps mezzanine with the fix here: https://dash.akamaized.net/WAVE/Mezzanine/under_review/2022-08-16/

Here is an overview of the mezzanine streams...

Latest mezzanine release:

Additional changes for next release (3):

yanj-github commented 1 year ago

Thanks @yanj-github, I thought you noticed the issue when using the test content, so I was wondering in which content you'd seen it.

Yes I have noticed issue when running tests on test runner. Please find detail here https://github.com/cta-wave/dpctf-tests/blob/dpcat/tests.csv Streams on date 2022-01-17 are used.

yanj-github commented 1 year ago

I uploaded new 30fps mezzanine with the fix here: https://dash.akamaized.net/WAVE/Mezzanine/under_review/2022-08-16/

I just done a random check of 2-3 stream on here they seems fine.

yanj-github commented 1 year ago

@nicholas-fr I have checked this on https://dash.akamaized.net/WAVE/Mezzanine/releases/3 just one file and it looks good to me. QR code at the end reports correct 01:00:00. I think this issue now is to regeneration of test content using lastest mezzanine and further link correctly to TR. if @rbouqueau and @louaybassbouss have this in the agenda already I will close this issue. Let me know if this is fine please? Just one sugguestion @nicholas-fr it would be better to remove the faulty mezzanine or moev it to to be deleted folder it would prevent futher faults.