-
Notifications
You must be signed in to change notification settings - Fork 2
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
Conversation
@takuro-Ishida 、初contributionの予感…! |
いやIssueとか作っとるから初contributionではないか |
#19 に書いてたわmarginalの話 |
Issue見過ごしてました... |
いやむしろこれが色々なこと書きすぎ感ありますね…一個づつ潰していきたいもんです |
@takuro-Ishida
お話で解決しました |
@forusufia この例は「屋外で遊ぶとき」で指定した時に回答者マップと飲料マップでのmarginal component planeを見た時にだいたい同じ結果になってます. |
There was a problem hiding this 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 の彩色の割合が大まかに一致している.
一度approve していましたが消えてしまいました? |
マージしまーす |
** Description 説明 **
#19 の1. Marginal Component Plane表示を行いました
Fixes #111
** Type of change 変更の種類**
** How Has This Been Tested? どのようにテストしたか?**
reviewerさんにお願いしたいこと
お忙しい中申し訳ありませんがよろしくお願いします.