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-12460: add ISO_IEC_13818-4 MPEG-2 AAC compressedAdif test suite #206

Merged
merged 5 commits into from
Nov 18, 2024

Conversation

rsanchez87
Copy link
Contributor

No description provided.

@rsanchez87 rsanchez87 marked this pull request as draft November 13, 2024 15:16
@rsanchez87 rsanchez87 force-pushed the COM-12460 branch 2 times, most recently from 34e73b8 to 186d1d2 Compare November 14, 2024 11:58
Base automatically changed from COM-12461 to master November 14, 2024 12:07
@rsanchez87 rsanchez87 closed this Nov 14, 2024
@rsanchez87 rsanchez87 reopened this Nov 14, 2024
…ogic to create JSON MPEG2_AAC-ADIF.json

* Create only MD5SUM ("result" in JSON) from "_f00" file if exists
* Improve download function, adding mutex and retries
* Add "-w" option to MPEG2 audio reference decoder command
@rsanchez87 rsanchez87 marked this pull request as ready for review November 15, 2024 12:34
@rsanchez87 rsanchez87 requested review from mdimopoulos and removed request for mdimopoulos November 15, 2024 12:35
Copy link
Contributor

@mdimopoulos mdimopoulos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of points to clarify.

fluster/utils.py Outdated Show resolved Hide resolved
test_suites/aac/MPEG2_AAC-ADIF.json Outdated Show resolved Hide resolved
scripts/gen_aac.py Outdated Show resolved Hide resolved
* remove debug prints
* fill source_checksum in case of not exists the test vector
@rsanchez87 rsanchez87 merged commit a6360e9 into master Nov 18, 2024
3 checks passed
@rsanchez87 rsanchez87 deleted the COM-12460 branch November 18, 2024 10:05
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