search

sayonari / twitchTransFreeNext

MIT License
222 stars 104 forks source link

add support for CeVIO #35

Closed yuniruyuni closed 2 years ago

yuniruyuni commented 2 years ago

What is this?

Why did you create this PR?

How implemented this PR?

Note

sayonari commented 2 years ago

すごい！ありがとうございます！ サポートは辞めてはいないです！時間がないだけで・・・． 色々と情報を頂き，大変嬉しいです！

config.pyの変数名が変わるのがちょっと気になりますが， 「バージョン上がったら，これまでのconfig.pyはそのままは使えないよ！」と言うようにします．

あと，Noteに書いていただいていた，pipするバージョンなどについては，僕も困っていたので， そこもぜひPRしてください！取り込ませてください！！！！！　おねがいします🙏m( )m

yuniruyuni commented 2 years ago

@sayonari -san. ありがとうございます～！サポートは最初に見かけたこのアプリのサイトに、サポート終了してますというのがあって気になってたのですが、該当のサイトはそもそも別の方が作った非公式のものだったみたいです。勘違いしてスイマセンm( )m

config.pyの変数名が変わるのがちょっと気になりますが，

あ、そういうことであればリネームとして同名のdeprecatedな変数を足して、「もし以前のものがあればwarnを出しながら、新変数に再代入する初期化」を書くというパターンも作れますがどうでしょうか…！(何れにしろUpdate時には変数名変更を推奨したほうがいい、とはなります)

sayonari commented 2 years ago

@yuniruyuni 先生： 変数追加＆warn対応のご提案を頂きありがとうございます！ そこまで頑張らなくても良い気がします．まあ，変数名が変わったら変わったで，どうにかなるでしょう． まあでも，実装して頂ければ，マージします！　ちょっと　今も仕事に追われているので，確認と実際のマージは数日お待ち下さい ｡ﾟ(ﾟ´Д｀ﾟ)ﾟ｡

yuniruyuni commented 2 years ago

はい～。ということでgTTS_In, gTTS_Outについて上のコード実装しておきました～！ :bow: requirements.txt のほうは後ほど別のPRとしてお送りします～！

ちょっと　今も仕事に追われているので，確認と実際のマージは数日お待ち下さい

おつかれさまです💦 こちらは(もちろん)ゆっくりで全然かまわないのでご自愛ください……！

yuniruyuni commented 2 years ago

FYI: #36 にてdependency all updateのPRをなげておきました。dependencyの更新だけではあるのですが、構造上かなり変えないといけなかったので、こちらとconflictするはずです。そして変更量を考えるとおそらくあちらが先にmergeされるほうが良いかと思います :bow: