SakuraProject / sakura-bot

SakuraBotです。
MIT License
9 stars 3 forks source link

Docker移行が放置されてる件 #74

Closed kousakirai closed 1 year ago

kousakirai commented 1 year ago

対応するならするではよやってほしいし時間かかるってわかってるならわざわざmainブランチにぶち込むな定期 頼むから後続の開発者が手を出せんくなるからmainブランチに加えた変更を途中放棄しないでくれ

tuna2134 commented 1 year ago

今テスト期間中 察してくれ

kousakirai commented 1 year ago

それならなおさらmainブランチに入れるなよ。俺今環境構築しようと思ったら書きかけであろうDockerfileのせいで積んでるんだが

tuna2134 commented 1 year ago

ん? k8s使う前提でPR作って、それをマージした。 だけどゴタゴタで変えないといかんから、、、

kousakirai commented 1 year ago

そもそも問題なく動くか検証してないものをmainに入れんなよ...

tuna2134 commented 1 year ago

検証したよ

tuna2134 commented 1 year ago

Minikubeでな?

kousakirai commented 1 year ago

いやいや、されてないでしょ。実際メモリ不足でマシン死んでたし

kousakirai commented 1 year ago

俺が言ってんのは実機環境で動くかどうかっていうのを検証しろよってことだよ

tuna2134 commented 1 year ago

検証したよ

tuna2134 commented 1 year ago

とっくの昔にな? minikube startでk8sを動かしたけど、

tuna2134 commented 1 year ago

予想外だったからね、、、 docker build -t sakurabot . docker run sakurabot で試してみて

kousakirai commented 1 year ago

いやだからさぁ...人の話聞いてる?マージする前に実機環境で動作に支障なく動くかどうかを検証してくれって言ってんだよ。 ローカル検証したって実機で動かんなんてことざらにあるのはわかってるやろ...

tuna2134 commented 1 year ago

いや やっていて、それで死んだらそれこそどうなん?

kousakirai commented 1 year ago

普通にやっててってどういうこと?頼むから理解してる前提で喋るのをやめてくれ。その文章だと君が何を指してるかすら理解ができん。

tuna2134 commented 1 year ago

それぐらい考えてくれ 実機環境でテストするだろ? それで死んだら、、、変わらんやん

tuna2134 commented 1 year ago

あとpipenvで今のところ動くはずだからそれでやってくれ

kousakirai commented 1 year ago

何を言ってんのかようわからんが、実機で動くかわからないものを自分のローカル環境で動いたからってmainブランチにPRして実際メモリ不足で鯖落としてるのは十分害悪行為やぞ。mainブランチにPRする前にk8sがどれぐらいのメモリを消費して、鯖側のメモリのサイズを把握する必要はあったんやない?

tuna2134 commented 1 year ago

鯖側のメモリーを把握するのは確かにすべきだったと思うが、そこまで少ないとは思わなかった。

kousakirai commented 1 year ago

まあもうこの話はいいや。Docker対応は俺が引継ぎするわ。幸いしばらく時間あるから動かせるレベルにはできそう。 ただ少し気になったのがyaakiyuの環境ってdocker動かせるんか? @yaakiyu

tuna2134 commented 1 year ago

まあもうこの話はいいや。Docker対応は俺が引継ぎするわ。幸いしばらく時間あるから動かせるレベルにはできそう。 ただ少し気になったのがyaakiyuの環境ってdocker動かせるんか? @yaakiyu

もうめんどいから削除した

kousakirai commented 1 year ago

dockerはあったほうが俺的にも開発楽にできるからどちらにせよやるのはやるよ。

tuna2134 commented 1 year ago

開発環境向けで頼んだ

yaakiyu commented 1 year ago

僕dockerの使い方わからんからこのissue丸投げなんだけど どうしようかな

tuna2134 commented 1 year ago

devcontainerセットアップでいいんじゃない?

tuna2134 commented 1 year ago

そもそもbot自体が特殊すぎる CUDAとかを必要とするし

yaakiyu commented 1 year ago

ま、よくわからんから引き続き丸投げで(

ysmreg commented 1 year ago

cudaはなくても動作しますがtorchが必要なのは機械学習させたpthをつかって読み上げさせようとしたためです。cuda がない場合はCpuで音声合成されます

tuna2134 commented 1 year ago

PyTorch自体デカすぎる

ysmreg commented 1 year ago

確かにpytorchは容量大きすぎますね()

yaakiyu commented 1 year ago

完了。