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

Added camera quality test using brisque (New) #759

Closed
wants to merge 8 commits into from

Conversation

fernando79513
Copy link
Collaborator

@fernando79513 fernando79513 commented Oct 11, 2023

Description

We want to have in place a basic test to check if the webcam signal is properly retrieved by the camera. The idea is to use an implementation of the BRISQUE algorithm to assess if the quality of the image is adequate and if we don't have issues with the decoding.

Resolved issues

Resolves "https://warthogs.atlassian.net/browse/CHECKBOX-456"

Documentation

So far, this PR does not include any changes in the documentation

Tests

The test is included in the Camera tests (automated). Alternatively, you can run the Python script under providers/base/bin/ called camera_quality_test.py

@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

Merging #759 (9a9b79c) into main (c3c3f45) will increase coverage by 0.08%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #759      +/-   ##
==========================================
+ Coverage   34.83%   34.92%   +0.08%     
==========================================
  Files         302      303       +1     
  Lines       34165    34211      +46     
  Branches     5909     5918       +9     
==========================================
+ Hits        11903    11949      +46     
  Misses      21697    21697              
  Partials      565      565              
Flag Coverage Δ
checkbox-support 49.73% <ø> (ø)
provider-base 3.44% <100.00%> (+0.31%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
providers/base/bin/camera_quality_test.py 100.00% <100.00%> (ø)

@fernando79513 fernando79513 marked this pull request as draft October 11, 2023 16:16
@fernando79513 fernando79513 changed the title DRAFT: Added camera quality test using brisque (New) Added camera quality test using brisque (New) Oct 11, 2023
@fernando79513 fernando79513 force-pushed the CHECKBOX-456/webcam-signal-test branch from 8cc0b39 to d65b611 Compare October 11, 2023 17:21
@@ -68,6 +69,7 @@ _description: Camera tests After Suspend (automated)
include:
after-suspend-camera/detect certification-status=blocker
after-suspend-camera/multiple-resolution-images_.* certification-status=blocker
camera/camera-quality_* certification-status=non-blocker
Copy link
Contributor

Choose a reason for hiding this comment

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

Prefix our job with after-suspend- here. This version of your checkbox camera job is automatically instantiated if the flag after-suspend- is used

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I have added the after-suspend flag

@fernando79513 fernando79513 force-pushed the CHECKBOX-456/webcam-signal-test branch 11 times, most recently from f4c459c to f743854 Compare October 20, 2023 11:44
@fernando79513 fernando79513 marked this pull request as ready for review October 23, 2023 09:46
@fernando79513 fernando79513 force-pushed the CHECKBOX-456/webcam-signal-test branch 5 times, most recently from 4cc892d to 56d7da6 Compare October 26, 2023 16:04
fernando79513 and others added 8 commits October 27, 2023 11:37
Added modified version of the brisque algorithm in
chechkbox_support/vendor.

The algorithm is implemented in:
https://github.com/rehanguha/brisque

Vendorized also python-libsvm to fix imports in snap
New camera quality test using brisque algorithm included in
"camera-cert-automated"
Included some automated tests to the image quality test to increase
the coverage
…indings

Nota: Only for 22.04 (until the syntax allows >= 22.04)
@fernando79513 fernando79513 force-pushed the CHECKBOX-456/webcam-signal-test branch from 56d7da6 to 9a9b79c Compare October 27, 2023 09:37
@fernando79513 fernando79513 deleted the CHECKBOX-456/webcam-signal-test branch October 27, 2023 13:48
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