Tidy up the types and logs - Githubissues

zerebom / H-and-M-Personalized-Fashion-Recommendations

(133rd/2952🥈)

4 stars 2 forks source link

Tidy up the types and logs #9

Closed zerebom closed 2 years ago

zerebom commented 2 years ago

Why

意図しない代入や型変換によるバグを避けたい。

What

以下の変更を加える

型の整備
- type hintingの追加
- 型に沿った変数名に変更
- 型の整備
  - feature_block: 入力: cudf, 出力: cudf
  - XXX_block: 入力: cudf, 出力: df
logger.infoの整備
- X_date, y_dateの出力
- Xに含まれる cust, art idのnuniquesの出力
- printの撲滅
その他
- グローバル変数になってるtrans_cdfを廃止
- 上記実装を追加した011_.pyの追加
- format by black

まだできていないところ

LBと相関したmapkの構築

zerebom commented 2 years ago

コピー、ファイル移動を除いた本質diff↓ https://github.com/zerebom/h_and_m/pull/9/files/d0f1758fd438c63db463231199e650ffc6f117d0..de65b93167b8a2bb73d97f137b7be52bb3543688#