search

nicholas-fr / test-content-validation

CTA WAVE Test Content Validation
BSD 3-Clause "New" or "Revised" License
1 stars 1 forks source link

test_stream_dir incorrectly set #2

Closed Lis-TV closed 2 years ago

Lis-TV commented 2 years ago

line 353, test_stream_dir = Path(str(str(tc_vectors_folder)+'\'+tc.file_brand[0]+TS_LOCATION_SETS_POST)

The problem is the tc.file_brand[0] does not correspond to the real folder names (avc_sets, caa_sets). Propose to change the -v call parameter to point to folder with t1..t34 subfolders.

nicholas-fr commented 2 years ago

I believe this issue is due to outdated test vectors, that use an old folder/file structure. The current agreed structure was discussed here and is:

_sets/// Example: https://dash.akamaized.net/WAVE/vectors/cfhd_sets/12.5_25_50/t1/2022-07-20/ If the folder structure changes again, then I will keep your proposal in mind as a fall back, to ensure the script works.
nicholas-fr commented 2 years ago

Thanks for raising this issue, I believe this has been explained/addressed. If you have further feedback, please feel free to reopen or create a new issue.