Skip to content

Fix issue #136: Remove dependency on speechrecognition library #137

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

neubig
Copy link
Contributor

@neubig neubig commented Jun 4, 2025

This PR fixes issue #136 by removing the dependency on the speechrecognition library.

Changes

  • Created a minimal implementation of the needed audio functionality in openhands_aci/utils/audio.py
  • Updated imports in md_converter.py to use our own implementation
  • Removed the speechrecognition dependency from pyproject.toml
  • Added unit tests for the new audio module
  • Added proper attribution for code adapted from the speech_recognition library (BSD 3-Clause License)
  • Fixed linting and type checking issues

Closes #136

@neubig can click here to continue refining the PR

Copy link

openhands-ai bot commented Jun 4, 2025

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • Run Python Unit Tests
    • Run Python Integration Tests

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #137

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

@neubig neubig requested a review from ryanhoangt June 6, 2025 04:54
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.

Remove dependency on speechrecognition library
2 participants