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

音声データを Ogg ファイルで出力できる機能の追加 #199

Merged
merged 11 commits into from
Jan 8, 2025

Conversation

Hexa
Copy link
Contributor

@Hexa Hexa commented Jan 6, 2025

This pull request includes several changes to enhance the functionality and flexibility of the audio processing system, specifically by adding support for saving audio data as Ogg files and handling additional header information.

Enhancements to audio processing:

  • amazon_transcribe_handler.go, packet_dump_handler.go, speech_to_text_handler.go, test_handler.go, service_handler.go: Modified the Handle method signatures to include a soraHeader parameter for handling additional header information. [1] [2] [3] [4] [5]
  • handler.go: Added a new soraHeader struct to encapsulate header information, and updated relevant methods to utilize this struct. [1] [2] [3]

Ogg file output support:

  • config.go, config_example.ini: Added new configuration options EnableOggFileOutput and OggDir to control the output of Ogg files and specify the directory for saving them. [1] [2] [3]
  • handler.go: Updated the opus2ogg function to support writing Ogg files to disk when enabled, and handle potential errors such as missing directories or permission issues. [1] [2]

Testing:

  • handler_test.go: Added new tests to verify the functionality of Ogg file writing, including scenarios for successful writes, disabled output, permission errors, and non-existent directories.

@Hexa Hexa requested a review from voluntas January 7, 2025 07:40
@Hexa Hexa marked this pull request as ready for review January 7, 2025 07:41
handler.go Outdated Show resolved Hide resolved
@Hexa
Copy link
Contributor Author

Hexa commented Jan 8, 2025

変更履歴が抜けていたため、CHANGES に追記しました

@Hexa Hexa merged commit 1800950 into develop Jan 8, 2025
1 check passed
@Hexa Hexa deleted the feature/add-ogg-file-output branch January 8, 2025 06:39
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