Closed sarrrrry closed 3 years ago
ディレクトリ構成及び必要な関数群の「外側だけ」の実装
prts 配下に置くとしていた、各種クラスを prts/time_series_metrics で1階層下に配置してみました。 理由は、interfaceクラスを実装した事により、同階層に配置される事に違和感があったからです。 このディレクトリ名(あるいは階層を掘るかどうか)、が独断で判断してしまったため、レビューをいただきたいです。
prts/time_series_metrics
make unittest make[1]: Entering directory '/home/rsato/Codes/github.com.sarrrrry/CompML/PRTS' poetry run python -m pytest --durations=0 --cov=prts tests ============================= test session starts ============================== platform linux -- Python 3.8.3, pytest-6.2.1, py-1.10.0, pluggy-0.13.1 rootdir: /home/rsato/Codes/github.com.sarrrrry/CompML/PRTS plugins: cov-2.10.1 collected 3 items
tests/test_precision.py . [ 33%] tests/test_precision_recall.py . [ 66%] tests/test_recall.py . [100%]
TOTAL 169 150 11%
============================== slowest durations ===============================
(9 durations < 0.005s hidden. Use -vv to show these durations.) ============================== 3 passed in 0.17s =============================== make[1]: Leaving directory '/home/rsato/Codes/github.com.sarrrrry/CompML/PRTS'
## Schedule None
@sarrrrry LGTM
Issue Number
6
Implementation Summary
ディレクトリ構成及び必要な関数群の「外側だけ」の実装
Scope of Impact
Particular points to check
prts 配下に置くとしていた、各種クラスを
prts/time_series_metrics
で1階層下に配置してみました。理由は、interfaceクラスを実装した事により、同階層に配置される事に違和感があったからです。 このディレクトリ名(あるいは階層を掘るかどうか)、が独断で判断してしまったため、レビューをいただきたいです。
Test
tests/test_precision.py . [ 33%] tests/test_precision_recall.py . [ 66%] tests/test_recall.py . [100%]
----------- coverage: platform linux, python 3.8.3-final-0 ----------- Name Stmts Miss Cover
prts/init.py 0 0 100% prts/interfaces/init.py 0 0 100% prts/interfaces/time_series_metrics.py 5 1 80% prts/prts.py 149 149 0% prts/time_series_metrics/init.py 0 0 100% prts/time_series_metrics/precision.py 5 0 100% prts/time_series_metrics/precision_recall.py 5 0 100% prts/time_series_metrics/recall.py 5 0 100%
TOTAL 169 150 11%
============================== slowest durations ===============================
(9 durations < 0.005s hidden. Use -vv to show these durations.) ============================== 3 passed in 0.17s =============================== make[1]: Leaving directory '/home/rsato/Codes/github.com.sarrrrry/CompML/PRTS'