yamashita-a420 / tight-upper-arm

0 stars 0 forks source link

Add: ER図の追加 #3

Closed yamashita-a420 closed 2 years ago

yamashita-a420 commented 2 years ago

ER図を作成しました。 お手数ですが、ご確認をお願いします。

https://gyazo.com/6236a08b81e4e7e8feedf372e6c1e704

懸念点:

  1. 現状考えている機能・計算方法は、この2テーブルでできると考えました。抜けがないか、わかる範囲でレビューいただけると幸いです。  画像処理方法:  https://narrow-mollusk-cd7.notion.site/Python-Numo-NArray-68b5f81a2a954020ba4e114eef70a1af  撮影方法:  https://narrow-mollusk-cd7.notion.site/4632d8a766ca44afae37400263c9f322

  2. バックエンドの機能をもう少し厚くする場合、考えられる機能などアドバイスいただけると嬉しいです。  案:表情の感情分析(笑顔の詳しい評価を返す)

murata0705 commented 2 years ago

画像処理はあまり触ったことが無いのでよくわかりませんが、処理できるという前提で言えば、テーブルはよさそうです 👍🏻

画面遷移図では length_standard を後から変更することができるように見えましたが、ここをあとから変更すると upper_arms の width と percentage_change を計算し直す必要があるかなと思ったので、その点は抑えておく必要があるかなと思いました。

等ですね。

バックエンドの機能をもう少し厚くする場合、考えられる機能などアドバイスいただけると嬉しいです。

どういう機能が考えられるかにもよりますが、「案:表情の感情分析(笑顔の詳しい評価を返す)」あたり検討するなら、画像は upper_arms に持たせず、もう少し抽象化した概念として持ったほうが良いかもしれないですね。

photo_records 的な(テーブル名は適当です)テーブルを用意しておいて、そこから upper_amrs とつなげるとかですかね。

photo_records - 中間テーブル - upper_arms
photo_records - 中間テーブル - emotion_ analysis
murata0705 commented 2 years ago

コメントしたので一応修正依頼にしておきますが、内容的には問題ないので、特に追加で質問なければ LGTM にします ✋🏻

murata0705 commented 2 years ago

バックエンドの機能をもう少し厚くする場合、考えられる機能などアドバイスいただけると嬉しいです。

あ、どういう機能があったらよいか?ですか?実装ではなくユーザーとしてどんな機能があったらよいかってことですかね?

yamashita-a420 commented 2 years ago

@murata0705 レビューありがとうございます!

画面遷移図では length_standard を後から変更することができるように見えましたが、ここをあとから変更すると upper_arms の width と percentage_change を計算し直す必要があるかなと思ったので、その点は抑えておく必要があるかなと思いました。 ・username のように最初しか設定できない ・変更を許可して、変更した際に再計算が走るようにしておく 等ですね。

変更を許可したいので、後者の方で実装しようと思います! 変更した際に再計算が走り、できればプレビュー表示してユーザー確認後、数値を更新してDB登録するように考えていきます。

photo_records 的な(テーブル名は適当です)テーブルを用意しておいて、そこから upper_amrs とつなげるとかですかね。

アドバイスありがとうございます!難しそうですが、感情分析機能をつける場合のER図も描いてみました。 emotion_analysisテーブルのtypeカラムは、このような理解であっていますか?

https://gyazo.com/6d79e978e40f095cf12ea4acfec5a219

あ、どういう機能があったらよいか?ですか?実装ではなくユーザーとしてどんな機能があったらよいかってことですかね?

欲張りな質問で、実装も機能も両方でした。 ただ実装のアドバイスも頂戴しましたし、先ほど技術面談で土屋講師に質問もできましたので大丈夫です! MVP後の追加実装として、ちょうど昨日アップされたRUNTEQのYouTube動画を参考に、(ツール系PFなので)友達などグループで使える仕様にする案ができました。

murata0705 commented 2 years ago

アドバイスありがとうございます!難しそうですが、感情分析機能をつける場合のER図も描いてみました。 emotion_analysisテーブルのtypeカラムは、このような理解であっていますか?

はい、良さそうです。

欲張りな質問で、実装も機能も両方でした。

個人的には成功した人の成長曲線みたいなものと自分の進捗を重ねて見れる機能がほしいですね。 下の記事は語学学習塾のサイトから引っ張ってきましたが、ダイエット系も成果が見えるまでに挫折しそうなので、成功者の軌跡と自分の現状の数値を重ねて、今どの辺にいるみたいなものが見れたら、成果が全然見えなくても、最初は成果が見えないものだから気長にやっていこうという気持ちになって、継続できるかなと思いました。

E68890E995B7E69BB2E7B79A-768x483

yamashita-a420 commented 2 years ago

@murata0705 ご確認いただきありがとうございます!こちらもなるべく実装する方向でいきたいと思います。

個人的には成功した人の成長曲線みたいなものと自分の進捗を重ねて見れる機能がほしいですね。

なるほど、とてもいいですね!それにはまず自分で成功モデルのデータを作る必要がありますね💪 アイデアありがとうございます!追加実装案で検討します!

今のところ質問は以上です。ご回答ありがとうございます!

murata0705 commented 2 years ago

あ、誰のレコードがわからなくなるので、photo_records は user_id が必要そうですね。 実装頑張ってください 🎉

yamashita-a420 commented 2 years ago

@murata0705 ほんとですね。もう少しつめて実装開始します。ありがとうございます!