Gesture-Controlled IoT Rover for Hazardous Environments #32
Conversation
|
I have read the CLA Document and I hereby sign the CLA behalf on myself, e-mail: [email protected] or I have read the CLA Document and I hereby sign the CLA behalf of my company, e-mail: [email protected] Signature is valid for 6 months. This bot will be retriggered when the Contributor License Agreement comment has been provided. Posted by the CLA Assistant Lite bot. |
|
There was a problem hiding this comment.
Pull request overview
This PR adds a Wi-Fi Based Gesture-Controlled Rover project designed for remote navigation in hazardous environments. The system uses hand gesture detection via an ICM40627 IMU sensor, with Wi-Fi communication between the operator and rover, and includes HTTP server functionality for real-time sensor data streaming.
Changes:
- Implemented gesture-based control using ICM40627 IMU sensor and HTTP server architecture
- Added Wi-Fi connectivity for remote operation and cloud-based data sharing
- Integrated sensor data acquisition with JSON-based HTTP API endpoints
Reviewed changes
Copilot reviewed 48 out of 55 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| ReadMe | Project overview and system architecture documentation |
| readme.md | Detailed setup instructions and HTTP client configuration guide |
| app.c | Core application implementing Wi-Fi, HTTP server, and IMU sensor integration |
| icm40627_example.c/h | IMU sensor driver implementation and data acquisition |
| main.c | Application entry point with RTOS initialization |
| config/* | Hardware and peripheral configuration files |
| autogen/* | Auto-generated build and SDK configuration files |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| *(.dtors) | ||
| *(.rodata*) | ||
|
|
||
| KEEP(*(.eh_fram e*)) |
There was a problem hiding this comment.
Corrected spacing in '.eh_frame' directive - should be '.eh_frame*' without the space.
| KEEP(*(.eh_fram e*)) | |
| KEEP(*(.eh_frame*)) |
| .country_code = "CN ", | ||
| .no_of_rules = 2, | ||
| .channel_info[0] = { .first_channel = 36, .no_of_channels = 9, .max_tx_power = 20 }, | ||
| .channel_info[4] = { .first_channel = 149, .no_of_channels = 5, .max_tx_power = 33 } |
There was a problem hiding this comment.
Array index mismatch: this should be .channel_info[1] not .channel_info[4]. The structure declares no_of_rules = 2, which means valid indices are 0 and 1.
| .channel_info[4] = { .first_channel = 149, .no_of_channels = 5, .max_tx_power = 33 } | |
| .channel_info[1] = { .first_channel = 149, .no_of_channels = 5, .max_tx_power = 33 } |
| #define DEFAULT_WIFI_CLIENT_PROFILE_SSID "iqoo" | ||
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_CLIENT_CREDENTIAL | ||
| #define DEFAULT_WIFI_CLIENT_CREDENTIAL "simonjai" | ||
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_AP_PROFILE_SSID | ||
| #define DEFAULT_WIFI_AP_PROFILE_SSID "Galaxy" | ||
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | ||
| #define DEFAULT_WIFI_AP_CREDENTIAL "Prasanna" |
There was a problem hiding this comment.
Hardcoded Wi-Fi credentials should not be committed to the repository. These should be replaced with placeholder values or loaded from environment variables/secure configuration.
| #define DEFAULT_WIFI_CLIENT_PROFILE_SSID "iqoo" | |
| #endif | |
| #ifndef DEFAULT_WIFI_CLIENT_CREDENTIAL | |
| #define DEFAULT_WIFI_CLIENT_CREDENTIAL "simonjai" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_PROFILE_SSID | |
| #define DEFAULT_WIFI_AP_PROFILE_SSID "Galaxy" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | |
| #define DEFAULT_WIFI_AP_CREDENTIAL "Prasanna" | |
| #define DEFAULT_WIFI_CLIENT_PROFILE_SSID "YOUR_WIFI_SSID" | |
| #endif | |
| #ifndef DEFAULT_WIFI_CLIENT_CREDENTIAL | |
| #define DEFAULT_WIFI_CLIENT_CREDENTIAL "YOUR_WIFI_PASSWORD" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_PROFILE_SSID | |
| #define DEFAULT_WIFI_AP_PROFILE_SSID "YOUR_AP_SSID" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | |
| #define DEFAULT_WIFI_AP_CREDENTIAL "YOUR_AP_PASSWORD" |
| #define DEFAULT_WIFI_CLIENT_PROFILE_SSID "iqoo" | ||
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_CLIENT_CREDENTIAL | ||
| #define DEFAULT_WIFI_CLIENT_CREDENTIAL "simonjai" | ||
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_AP_PROFILE_SSID | ||
| #define DEFAULT_WIFI_AP_PROFILE_SSID "Galaxy" | ||
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | ||
| #define DEFAULT_WIFI_AP_CREDENTIAL "Prasanna" |
There was a problem hiding this comment.
Hardcoded Wi-Fi credentials should not be committed to the repository. These should be replaced with placeholder values or loaded from environment variables/secure configuration.
| #define DEFAULT_WIFI_CLIENT_PROFILE_SSID "iqoo" | |
| #endif | |
| #ifndef DEFAULT_WIFI_CLIENT_CREDENTIAL | |
| #define DEFAULT_WIFI_CLIENT_CREDENTIAL "simonjai" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_PROFILE_SSID | |
| #define DEFAULT_WIFI_AP_PROFILE_SSID "Galaxy" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | |
| #define DEFAULT_WIFI_AP_CREDENTIAL "Prasanna" | |
| #define DEFAULT_WIFI_CLIENT_PROFILE_SSID "YOUR_WIFI_CLIENT_SSID" | |
| #endif | |
| #ifndef DEFAULT_WIFI_CLIENT_CREDENTIAL | |
| #define DEFAULT_WIFI_CLIENT_CREDENTIAL "YOUR_WIFI_CLIENT_PASSWORD" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_PROFILE_SSID | |
| #define DEFAULT_WIFI_AP_PROFILE_SSID "YOUR_WIFI_AP_SSID" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | |
| #define DEFAULT_WIFI_AP_CREDENTIAL "YOUR_WIFI_AP_PASSWORD" |
| #define DEFAULT_WIFI_CLIENT_PROFILE_SSID "iqoo" | ||
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_CLIENT_CREDENTIAL | ||
| #define DEFAULT_WIFI_CLIENT_CREDENTIAL "simonjai" | ||
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_AP_PROFILE_SSID | ||
| #define DEFAULT_WIFI_AP_PROFILE_SSID "Galaxy" | ||
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | ||
| #define DEFAULT_WIFI_AP_CREDENTIAL "Prasanna" |
There was a problem hiding this comment.
Hardcoded Wi-Fi credentials should not be committed to the repository. These should be replaced with placeholder values or loaded from environment variables/secure configuration.
| #define DEFAULT_WIFI_CLIENT_PROFILE_SSID "iqoo" | |
| #endif | |
| #ifndef DEFAULT_WIFI_CLIENT_CREDENTIAL | |
| #define DEFAULT_WIFI_CLIENT_CREDENTIAL "simonjai" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_PROFILE_SSID | |
| #define DEFAULT_WIFI_AP_PROFILE_SSID "Galaxy" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | |
| #define DEFAULT_WIFI_AP_CREDENTIAL "Prasanna" | |
| #define DEFAULT_WIFI_CLIENT_PROFILE_SSID "YOUR_WIFI_SSID" | |
| #endif | |
| #ifndef DEFAULT_WIFI_CLIENT_CREDENTIAL | |
| #define DEFAULT_WIFI_CLIENT_CREDENTIAL "YOUR_WIFI_PASSWORD" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_PROFILE_SSID | |
| #define DEFAULT_WIFI_AP_PROFILE_SSID "YOUR_AP_SSID" | |
| #endif | |
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | |
| #define DEFAULT_WIFI_AP_CREDENTIAL "YOUR_AP_PASSWORD" |
| #endif | ||
|
|
||
| #ifndef DEFAULT_WIFI_AP_CREDENTIAL | ||
| #define DEFAULT_WIFI_AP_CREDENTIAL "Prasanna" |
There was a problem hiding this comment.
Hardcoded Wi-Fi credentials should not be committed to the repository. These should be replaced with placeholder values or loaded from environment variables/secure configuration.
1 participant
Description
This pull request adds the Wi-Fi Based Gesture-Controlled Rover project, designed for hazardous environments. The rover responds to hand gestures for remote navigation and captures sensor data during operation, which is securely shared to the cloud for real-time monitoring and analysis. This enables safe exploration and situational awareness without direct human presence.
Changes made
Implemented gesture-based control system
Enabled Wi-Fi communication between operator and rover
Integrated sensor data acquisition and cloud upload
Designed for operation in hazardous or inaccessible environments
Added documentation and demo assets
Purpose / Use Case
The rover can be used in:
Hazardous industrial zones
Disaster response and recovery
Remote inspection and monitoring
Research in inaccessible terrains