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

Feat/#51 #52

Merged
merged 6 commits into from
May 3, 2024
Merged

Feat/#51 #52

merged 6 commits into from
May 3, 2024

Conversation

aiueo-1234
Copy link

TaPetさんのパソコンで作業していたのでコミットがTaPetさんになっていますが、作業者は自分です

@aiueo-1234 aiueo-1234 requested a review from miyaji255 May 2, 2024 14:20
@aiueo-1234 aiueo-1234 linked an issue May 2, 2024 that may be closed by this pull request
Comment on lines 26 to 28
var ms = new MemoryStream();
ms.Write(wavData);
ms.Position = 0;

Choose a reason for hiding this comment

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

usingつけておいてください。
また、コンストラクタでbyte[]が指定できたと思います。

@miyaji255
Copy link

miyaji255 commented May 3, 2024

レビューが遅れてしまいすみません。簡単に修正できるポイントだけコメントしました。結合した音声がうまく再生できていることは確認できましたか?

@aiueo-1234
Copy link
Author

統合した音声を使って作られたepubがdiscordに上がっています。その音声を聞く限り音声統合はうまく行っていると思います

@miyaji255
Copy link

了解しました。修正はできそうですか?忙しいならこちらで修正します

@aiueo-1234
Copy link
Author

了解しました。修正はできそうですか?忙しいならこちらで修正します

今から修正します

@aiueo-1234 aiueo-1234 requested a review from miyaji255 May 3, 2024 02:08
miyaji255
miyaji255 previously approved these changes May 3, 2024
Copy link

@miyaji255 miyaji255 left a comment

Choose a reason for hiding this comment

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

大体OKだと思います。

@miyaji255 miyaji255 enabled auto-merge May 3, 2024 02:11
@miyaji255 miyaji255 merged commit dd4a5d2 into main May 3, 2024
2 checks passed
@miyaji255 miyaji255 deleted the feat/#51 branch May 3, 2024 02:17
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.

Style-Bert-Vit2の音声出力周りの修正
3 participants