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

Implement tsom2 marginalcomp #112

Merged
merged 16 commits into from
Apr 2, 2020
Merged

Conversation

takuro-Ishida
Copy link
Member

@takuro-Ishida takuro-Ishida commented Dec 5, 2019

** Description 説明 **
#19 の1. Marginal Component Plane表示を行いました

Fixes #111

** Type of change 変更の種類**

  • New feature (non-breaking change which adds functionality)

** How Has This Been Tested? どのようにテストしたか?**

  • tutorials/tsom/beverage_data/How_to_use_tsom_and_Viewer.pyを動かし、古川研HPにある飲料データとの結果と一致するかを確認

reviewerさんにお願いしたいこと

  • 古川研HPにある飲料データのマップと仕様が同じであることを確認

お忙しい中申し訳ありませんがよろしくお願いします.

@takuro-Ishida takuro-Ishida self-assigned this Dec 5, 2019
@takuro-Ishida takuro-Ishida added the enhancement New feature or request label Dec 5, 2019
@takuro-Ishida takuro-Ishida removed the request for review from ae14watanabe December 5, 2019 14:38
@ae14watanabe
Copy link
Member

@takuro-Ishida 、初contributionの予感…!

@ae14watanabe
Copy link
Member

いやIssueとか作っとるから初contributionではないか

@ae14watanabe
Copy link
Member

#19 に書いてたわmarginalの話

@takuro-Ishida
Copy link
Member Author

Issue見過ごしてました...
時間がある時にでも #19 の仕様でやろうかなと思います

@ae14watanabe
Copy link
Member

いやむしろこれが色々なこと書きすぎ感ありますね…一個づつ潰していきたいもんです

@forusufia
Copy link
Contributor

forusufia commented Dec 10, 2019

@takuro-Ishida
選択、選択解除といったボタンの機能は確認できました。

テストで飲料データを使っているので、ユーザーとシチュエーションの組み合わせでのマップも見たほうが良いのでしょうか。
ユーザーと飲料の2面マップを見て、これで良しと判断してよいのかわかりません。

お話で解決しました

@takuro-Ishida
Copy link
Member Author

takuro-Ishida commented Dec 10, 2019

@forusufia
Issueにあげているものをこちらの方にあげています.
自分としては、somfのTSOM2で飲料データを動かしてもらって、situationを固定した時にユーザーマップと飲料マップでのmarginal component planeの結果を比較してももらえたらなと思っています.
期待する結果としては古川研のHPの結果と「おおむね」あっていればいいかなと思っています.

この例は「屋外で遊ぶとき」で指定した時に回答者マップと飲料マップでのmarginal component planeを見た時にだいたい同じ結果になってます.

image

image

@forusufia
Copy link
Contributor

「食事中」
一致してそうです!

6

7

Copy link
Contributor

@Kusumoto-Keisuke Kusumoto-Keisuke left a comment

Choose a reason for hiding this comment

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

以下の項目が確認できたので Approve します.

  • Beverages Map 上の飲み物の位置関係が古川研HPのものとほとんど一致している.
  • Situations(ラジオボタン) を選択したときの Users Map と Beverages Map の Marginal Component Plane が古川研HPのものとほとんど一致している
  • Situations(ラジオボタン)と Beverages Map の飲料を選択したときの Users Map の彩色の割合が大まかに一致している.

@forusufia
Copy link
Contributor

一度approve していましたが消えてしまいました?
もう一度approve しておきますね

@takuro-Ishida
Copy link
Member Author

マージしまーす

@takuro-Ishida takuro-Ishida merged commit 1abb81a into master Apr 2, 2020
@takuro-Ishida takuro-Ishida deleted the implement_tsom2_marginalcomp branch April 2, 2020 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TSOM2ビューワーにmarginal component planeの追加
4 participants