Repository navigation
Conversation
… on all platforms
- Fix FetchContent timestamp policy (CMP0135) in CMakeLists.txt - Implement robust rate-limiting tests in test_auth_logic.cpp - Modernize C++ usage (headers, casts, const) across PAM and service - Extract IPC protocol to src/common/ - Deep clean of build configuration
- **Build**: Disabled excessive GTest installation in CMake to resolve package conflicts. - **Tests**: Increased lockout duration in test_auth_logic.cpp to prevent flakiness on slower emulated architectures. - **Docker**: Fixed FetchContent SSL errors (added ca-certificates) and ensure models are correctly installed before running tests. - **Cleanup**: Updated .gitignore to exclude log files.
…dels - **CI**: Added step to , , and jobs to ensure tests running on host can find models in . - **CI**: Added missing step to . - **Models**: Updated and to fetch models from Hugging Face (OpenCV Zoo mirror). - **Models**: Updated YuNet model to .
…face detection, and include Git hash or build date in the version string.
- Promoted isValidUsername to public utility in utils.hpp - Removed unused headers (numeric, string_view) - Fixed CMake CMP0135 warning for googletest - Resolved test linking errors
- Updated Dockerfiles to install models to /usr/share/linuxcampam/models - Updated .dockerignore to exclude opencv_static and build artifacts to prevent cross-arch contamination
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR addresses the "AUTH_FAIL: No face detected" error during video calls by refining camera timing and timeouts. It also includes cross-platform build fixes and enhanced debugging output.
Fixes
IR_TRIGGER_DELAY_MSandCAMERA_WARMUP_FRAMESto ensure the IR camera has time to settle auto-exposure even when the RGB camera is busy.linuxcampam testnow reliably saves debug images (success and fail) for analysis.Type of change
How Has This Been Tested?
linuxcampam testcaptures valid IR images while an external process (video call) is using the RGB camera.i386,aarch64, andriscv64using the build script.Checklist