search

Qulacs-Osaka / scikit-qulacs

scikit-qulacs is a library for quantum neural network. This library is based on qulacs and named after scikit-learn.
https://qulacs-osaka.github.io/scikit-qulacs/index.html
MIT License
21 stars 6 forks source link

Clean up code #139

Closed ikanago closed 2 years ago

ikanago commented 2 years ago

close #136 動作を変えるような変更はしていません．修整漏れがあったら教えてください．

コードを眺めていて，入出力のスケーリング，コスト関数，ソルバはクラスとして切り出した方が見通しがよくなると思いました． 特にスケーリングのコードが重複しており意図が読み取りにくい状態なので，そこから取り掛かるのがよさそうです．

kosukemtr commented 2 years ago

@WATLE y_exp_ratio というのはどういう役割を持ったパラメータでしょうか？docstringの https://github.com/Qulacs-Osaka/scikit-qulacs/blob/0087024c24202eba5c98d9284aab8555a5db8430/skqulacs/qnn/classifier.py#L33 の部分です。

WATLE commented 2 years ago

softmaxで、たとえばデータAが0,データBが1,y_exp_ratioが3のとき、 Aが出る確率はe^0=1,に比例、Bが出る確率はe^3=20.08に比例 だからAは5%,Bは95%

kosukemtr commented 2 years ago

見た感じでコメントがたりていなかったところ、正確でなかったところのコメントを追加しました