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.

Sign up for GitHub

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

Jump to bottom

x_sigma=0.1@fitting_kura.py : 宣言するだけで使われてないと思うのですが... #134

Closed
kz-halfmoon opened this issue Apr 14, 2020 · 4 comments · May be fixed by #135
Closed

x_sigma=0.1@fitting_kura.py : 宣言するだけで使われてないと思うのですが... #134

kz-halfmoon opened this issue Apr 14, 2020 · 4 comments · May be fixed by #135
Labels
invalid This doesn't seem right question Further information is requested

Comments

@kz-halfmoon
Copy link

target code 対象となるコード
somf/tutorials/som/kura_data/fitting_kura.py

Is your feature request related to a problem? Please describe. この機能追加リクエストは何かの問題に関連していますか？
fitting_kura.pyの15行目の記述：x_sigma = 0.1
これは，宣言するだけで使われていないと思うのですが，理解はあっているでしょうか．
Describe the solution you'd like 望む解決策の説明
該当コードx_sigma = 0.1の削除

Additional context 追加の説明
宮崎さんに聞いたら，Issue立てといたほうがいい，とアドバイスしてくれたので立ててみました．
@kz-halfmoon kz-halfmoon added invalid This doesn't seem right question Further information is requested labels Apr 14, 2020
@ae14watanabe
Copy link
Member

ae14watanabe commented Apr 14, 2020 •
edited
Loading

使われてないですね。これはウッカリですね。本当は26行目の0.1 のところにx_sigmaを入れたかったんだと思う。要するにデータに乗っけるガウスノイズの標準偏差をコードの頭で宣言したかったんですね。変数名 x_sigma っていうのもいまいち分かりにくいね。

somf/tutorials/som/kura_data/fitting_kura.py

Line 26 in 1abb81a

X += np.random.normal(0,0.1,X.shape)

@kz-halfmoon
Copy link
Author

なるほど，そこの0.1でしたか．
観測空間のノードに乗せるノイズですかね？
素人目ですが，近傍半径に関わる変数かなと思ってました．

@ae14watanabe ae14watanabe mentioned this issue Apr 15, 2020
Open
@ae14watanabe
Copy link
Member

@kz-halfmoon
観測空間の データ に乗せるノイズですね。（ちなみに潜在空間にある代表点をノード、観測空間にあるノードの写像先を参照ベクトルと呼んでます。）
それがどれだけ大きいノイズにするのかをコントロールする変数ですね。

ノイズを乗せない場合は、滑らかな多様体の上に完全に乗ったデータになります。
ノイズを乗せた場合には多様体からちょっとはみ出してしまいます。

なぜノイズを乗せるかというと、、、後で説明します。

@ae14watanabe
Copy link
Member

@kz-halfmoon
あ，これ答えてないやん笑
要するにデータに対して仮定してる生成過程をトレースしてるわけね．
真の多様体からノイズが乗ってデータが生成されるでしょうという．

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
invalid This doesn't seem right question Further information is requested
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

宣言された変数を利用していないミスをfix furukawa-laboratory/somf
2 participants
@ae14watanabe @kz-halfmoon