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

be more explicit about camera requirements #81

Open
jpiesing opened this issue Feb 22, 2024 · 2 comments
Open

be more explicit about camera requirements #81

jpiesing opened this issue Feb 22, 2024 · 2 comments
Labels
documentation Improvements or additions to documentation

Comments

@jpiesing
Copy link
Contributor

Currently we just have one line about camera requirements.

camera that records at 120 fps or more (AVC/h.264)

Javier found another one - the OF only supports 48kHz audio and rejects 44.1kHz audio with a fatal error.

We may consider a new section on camera requirements perhaps something like this.

Camera requirements

The camera needs to be capable of recording video at 120Hz in AVC. It needs to be capable of recording audio at 48kHz. When getting started with the WAVE device test suite, the following may be useful purely for experimentation and familiarization with the test runner and test suite.

  • It is possible to use cameras only recording at 60Hz if you limit yourself to experimenting with those tests where the video frame rate is 25/30Hz.
  • If the camera does not record video in AVC then it is possible to use tools like ffmpeg to transcode to AVC however this is time-consuming and ffmpeg is not the easiest tool to use.
  • If the camera records audio at 44.1 kHz and not 48kHz then either ffmpeg can be used to remove the 44.1kHz audio track or it may be possible to disable the camera recording audio. Disabling the microphone may only result in silence being recorded at 44.1kHz. Removing the audio would mean limiting yourself to experimenting with a subset of tests.

@yanj-github Do you agree with the above?

@jpiesing jpiesing added the documentation Improvements or additions to documentation label Feb 28, 2024
@yanj-github
Copy link
Collaborator

@jpiesing document for audio testing camera requirement is in progress and we will commit it shortly.
Re;
Javier found another one - the OF only supports 48kHz audio and rejects 44.1kHz audio with a fatal error.
I have removed fatal error and only present warning when processing video tastings but it will still cause a fatal error when the process sees the audio or combined tests.
We will also look into your suggestions and update document accordingly.

@yanj-github
Copy link
Collaborator

yanj-github commented Mar 15, 2024

Camera requirements:
The camera needs to be capable of recording video at around 120Hz, full HD. It needs to be capable of wired connection from DUT to record audio at 48kHz, 16-bits jointly with video. When getting started with the WAVE device test suite, the following may be useful purely for experimentation and familiarization with the test runner and test suite.

  • It is possible to use cameras only recording at 60Hz if you limit yourself to experimenting with those tests where the video frame rate is 25/30Hz.
  • If the camera records audio at 44.1 kHz and not 48kHz then either ffmpeg can be used to remove the 44.1kHz audio track or it may be possible to disable the camera recording audio. Disabling the microphone may only result in silence being recorded at 44.1kHz. Removing the audio would mean limiting yourself to experimenting with video only test and video observations for video and audio combined tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants