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

ランダム行動観測ログに音声モダリティを追加 #187

Merged
merged 6 commits into from
Nov 10, 2024

Conversation

Geson-anko
Copy link
Member

概要

#186

ホストOSのサウンド系がPulseAudioであることを前提としています。
また、リモートデスクトップでオーディオデバイスが切り替わると音声が取得できなくなってしまうため、データ収集中は注意が必要です。

変更内容

影響範囲

Submit前の確認項目

  • タイトルは一目でわかるようにし、説明文は PR を簡潔に説明するようにしましたか?
  • あなたの PR が、異なる変更を束ねたものではなく、ひとつのことだけを行うものであることを確認しましたか?
  • この PR で導入されたすべての変更点をリストアップしましたか?
  • PR をmake runコマンドでローカルでテストしましたか?

補足

Sorry, something went wrong.

@Geson-anko Geson-anko linked an issue Nov 10, 2024 that may be closed by this pull request
1 task
@@ -1,4 +1,7 @@
"""ここではconfigファイル上からオブジェクトを正常にインスタンス化可能かテストします。"""
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"""ここではconfigファイル上からオブジェクトを正常にインスタンス化可能かテストします。"""
"""This test checks whether object can be successfully instantiated from config file."""

Copy link
Member Author

Choose a reason for hiding this comment

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

これは今回の変更に取り込むべきではないね。ということで少し直しておいた。6887cb4

@Geson-anko Geson-anko merged commit 28c3fe1 into main Nov 10, 2024
2 checks passed
@Geson-anko Geson-anko deleted the feature/#186/audio-log branch November 10, 2024 11:43
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.

音声モダリティも含めたRandomActionObservationLog
2 participants