search

furukawa-laboratory / somf

Python codes of SOM-family launched by Furukawa-laboratory, Kyushu institute of technology
5 stars 2 forks source link

宣言された変数を利用していないミスをfix #135

Open ae14watanabe opened 4 years ago

ae14watanabe commented 4 years ago

close #134 放物双曲面状に分布するデータに対してSOMでfittingするチュートリアルコードにおいて、宣言されていたのに利用されていない変数があったので、ちゃんと使うように変更しました。 それからついでに変数名も変更しました。

reviwerにお願いしたいこと 変更前と変更後で実行結果が変わらないことを確認してください。

ae14watanabe commented 4 years ago

@hirowatari-s @kz-halfmoon @nakashima1125 3人をレビュワーに立ててみました。とても簡単な変更なのですぐチェック終わると思います。Githubの使い方の練習だと思ってreviewしてみてください。質問があったらまた聞いてください。

ae14watanabe commented 4 years ago

チェックのためにはこのプルリクで出してるリモートブランチをローカルでチェックアウトする（言葉遣いが合ってるのか分からんけど…リモートブランチに追従するローカルブランチを作るって言えばいいのか）必要があります。 以下のブログの手順でできます。 https://www.setoya-blog.com/entry/2012/11/04/132746

ae14watanabe commented 4 years ago

@hirowatari-s @kz-halfmoon あざっす！ mergeするには十分なんですが、 @nakashima1125 もぜひやってみて