fit_model.py

[y-abe]

https://github.com/HiromuMasuda/gunosy_assignment/blob/master/classification_machine/management/commands/fit_model.py#L7-L9

• インポート時は基本的に * は使わず、必要なものだけにしたほうが良いですね。
• TfidfVectorizer は使っていないので、外してしまいましょうか。

https://github.com/HiromuMasuda/gunosy_assignment/blob/master/classification_machine/management/commands/fit_model.py#L64 n は数値を想起させるので、 name とかですかね。

https://github.com/HiromuMasuda/gunosy_assignment/blob/master/classification_machine/management/commands/fit_model.py#L75-L76 ファイル名はハードコードするのではなく、設定ファイルに書く等しましょう

https://github.com/HiromuMasuda/gunosy_assignment/blob/master/classification_machine/management/commands/fit_model.py#L31 N もハードコードではなくて、コマンドライン引数などで外側から渡せたほうがいいですね

[HiromuMasuda]

コメントありがとうございます。

https://github.com/HiromuMasuda/gunosy_assignment/pull/27 で修正しました。

[HiromuMasuda]

https://github.com/HiromuMasuda/gunosy_assignment/blob/master/classification_machine/management/commands/fit_model.py#L75-L76 ファイル名はハードコードするのではなく、設定ファイルに書く等しましょう

こちらは、#25で修正しています。