Skip to content

Added basic Voice Activity Detection (VAD) functionality#3

Merged
ducroq merged 10 commits intomainfrom
dev_VAD
Apr 14, 2025
Merged

Added basic Voice Activity Detection (VAD) functionality#3
ducroq merged 10 commits intomainfrom
dev_VAD

Conversation

@ducroq
Copy link
Contributor

@ducroq ducroq commented Apr 13, 2025

This PR adds an energy-based Voice Activity Detection (VAD) node to the ROS2_Audio_Tools package. For more advanced detection in challenging environments, consider integrating a machine learning-based VAD algorithm.

Features Added

  • New VoiceActivity.msg message type for publishing detection results
  • Energy-based VAD node implementation with configurable sensitivity
  • Launch file for easy testing of the VAD functionality
  • Comprehensive documentation including parameter tuning guide

Implementation Details

  • Simple but effective energy-based algorithm
  • Support for all existing audio formats
  • Configurable threshold, hold time, and minimum sample requirements
  • Real-time diagnostic information in published messages

Testing Done

  • Tested with various audio inputs and parameter configurations
  • Verified detection accuracy with different speech patterns
  • Confirmed parameter adjustment functionality

Documentation

  • Added VAD node details to implementation.md
  • Created voice_activity_detection.md with tuning instructions
  • Updated overview.md to include VAD functionality
  • Updated mkdocs.yml navigation

ducroq added 2 commits April 13, 2025 13:14
- Implement energy-based VAD node
- Add VoiceActivity message type
- Create launch file for audio capture with VAD
- Update documentation with VAD usage instructions and parameters
@Hoog-V Hoog-V linked an issue Apr 13, 2025 that may be closed by this pull request
Copy link
Member

@Hoog-V Hoog-V left a comment

Choose a reason for hiding this comment

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

Fix the above listed things and the needed things to pass workflow checks, because it will break things if I merge it right now. That's why the checks fail :)

Copy link
Member

@Hoog-V Hoog-V left a comment

Choose a reason for hiding this comment

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

It looks great, when checks done it will be ready to merge

@ducroq ducroq merged commit b7e64be into main Apr 14, 2025
4 checks passed
@ducroq ducroq deleted the dev_VAD branch April 14, 2025 07:26
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.

Add vad Node

3 participants