search

shigeyukioba / matchernet

Apache License 2.0
1 stars 1 forks source link

logging の実装 #2

Closed uchihashikenshi closed 4 years ago

uchihashikenshi commented 4 years ago

概要

print1, 2, 3, 4, 5 で実装されているものを logging を用いて再実装したい

目的

print で出力されたものがそのカテゴリに属するどの程度重要なメッセージなのか、使用者にわかりやすい logging という形で提示する。 あまり踏み込まない予定だが、ログ出力に関しては諸説ある。 参考: ログ出力のための print と import logging はやめてほしい

提案内容

import logging

logging.critical('critical')
logging.error('error')
logging.warning('warning')
logging.info('info')
logging.debug('debug')

のようなものを使う。可能な限り formatter を定義し、見え方が統一されるようにする。

タスク

まずは現状 pring debug で行いたい意図を理解し、https://github.com/shigeyukioba/matchernet_dev/blob/master/matchernet_py_001/utils.py にある print をすべて消しても同様の動作をすることを確認する。

関連

shigeyukioba commented 4 years ago

優先度は低いですけど、ぜひお願いします！

ktnyt commented 4 years ago

Production Codeに不要な出力関連のコードを残さないためにもLoggingは結構重要だったりします。

uchihashikenshi commented 4 years ago

見えた追加の改善点は以下。