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

[Enhancement] Move camera_quality_test.py to checkbox-support for other providers utilizing it #1667

Open
baconYao opened this issue Dec 20, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@baconYao
Copy link
Contributor

baconYao commented Dec 20, 2024

Enhancement Proposal

Reason

In many cases, due to various factors such as MIPI camera NDA, proprietary drivers, and others, many camera-related test items need to be implemented within each provider.

These vendor-specific tests make it challenging to integrate them into the base provider.

Additionally, it is often difficult to use the test cases in the base provider smoothly
- id: camera/roundtrip-qrcode_{{ name }}

  • Be used for almost all IoT devices
  • scanning a QR code requires a display/monitor to show the QR code

Since these vendor-specific tests focus on the output of camera and input of SoC can negotiate with different resolutions, colorspaces, formats, etc., (MediaTek with OnSemi AP1302+AR0830 or MediaTek with Sony IMX214), relying solely on manual judgment results in a heavy burden and cannot be effectively verified during SRU.

After reviewing the content of camera_quality_test.py, I believe this script could be adapted in the following two ways to bring greater benefits to everyone:

  • Move it from the base provider to checkbox-support.
    • Advantage: All providers can use the same set of validation standards, significantly aiding the automation of vendor-specific case validation.
  • Expand its capabilities to validate not only single photos but also video files.
    • Advantage: Whether it's a video or a photo, this method can be used for validation.
@baconYao baconYao added the enhancement New feature or request label Dec 20, 2024
Copy link

Thank you for reporting your feedback to us!

The internal ticket has been created: https://warthogs.atlassian.net/browse/CHECKBOX-1715.

This message was autogenerated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants