cta-wave / dpctf-tests

Repo for DPCTF Tests. We prefer to keep the Tests separated from Test Runner
Other
1 stars 5 forks source link

Configure required tolerance to A/V sync #99

Closed yanj-github closed 1 year ago

yanj-github commented 1 year ago

Re; the A/V sync observation that is defined in device capability spec as follows: "The mediaTime of the presented audio sample matches the one reported by the video currentTime value within the tolerance of +40 mS / -120 mS. "

Can we defined the tolerance in test-config.json file please?

louaybassbouss commented 1 year ago

@yanj-github yes but we need more details. What are the names of the parameter? For which tests?

yanj-github commented 1 year ago

A/V sync observation applys to all section 9 tests. As they are all same can we set this on section for "all", let me know if there is any objection please? I suggest to set value as follows: e.g: "av_sync_tolerance": [40, -120],

FritzHeiden commented 1 year ago

Added the parameter to the test-config

yanj-github commented 1 year ago

Thanks @FritzHeiden I can see that the av_sync_tolerance are definied now.