Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

COM-12449: Add H264 FRExt test suite generator #201

Merged
merged 2 commits into from
Nov 11, 2024
Merged

Conversation

mdimopoulos
Copy link
Contributor

  • implemented FRExt test suite generator code
  • slightly modified json file of FRExt test suite

- implemented FRExt test suite generator code
- slightly modified json file of FRExt test suite
- undo naming changes for H264 Professional profiles and SVC suites
@rsanchez87
Copy link
Contributor

rsanchez87 commented Nov 11, 2024

All changes look good and it's working,

Just a minimal comment @mdimopoulos , in the script gen_jvt.py, line 300, Do you think that to continue with the standardisation, it would be possible to rename this ts to "AVC" without v1?

@mdimopoulos
Copy link
Contributor Author

All changes look good and it's working,

Just a minimal comment @mdimopoulos , in the script gen_jvt.py, line 300, Do you think that to continue with the standardisation, it would be possible to rename this ts to "AVC" without v1?

Unfortunately we cannot do that because that parameter value is used to construct the URL (https://www.itu.int/wftp3/av-arch/jvt-site/draft_conformance/AVCv1/) where the files of the test suite are located. If i remove v1 it will start failing.
The other test suite don't have v1 because the URL doesn't include it (see https://www.itu.int/wftp3/av-arch/jvt-site/draft_conformance/)

@mdimopoulos
Copy link
Contributor Author

@rgonzalezfluendo BC issues fixed in latest commit.

@mdimopoulos mdimopoulos merged commit d35dae3 into master Nov 11, 2024
3 checks passed
@mdimopoulos mdimopoulos deleted the COM-12449 branch November 11, 2024 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants