adhit-r/fairmind

Align plot_numeric_distributions module path with tests

開放

#146 建立於 2026年1月6日

 (1 則留言) (0 個反應) (0 位負責人)Python (12 個分叉)auto 404
buggood first issuetesting

倉庫指標

星標
 (9 顆星)
PR 合併指標
 (PR 指標待抓取)

描述

Summary

The new plot_numeric_distributions helper added in PR #145 lives under apps/ml/visualizations/data_distribution.py, but the associated tests in tests/test_distribution.py currently import from a backend utilities path. This causes pytest to fail with ModuleNotFoundError once tests are run against main.

Tasks

  • Decide the canonical location for the data distribution utility:
    • Either keep it under apps/ml/visualizations/data_distribution.py, or
    • Move it under a backend utilities namespace (e.g. apps/backend/utils/visualization/data_distribution.py).
  • Update tests/test_distribution.py to import plot_numeric_distributions from the chosen canonical module path.
  • Run pytest tests/test_distribution.py and ensure the tests pass.

Acceptance Criteria

  • pytest tests/test_distribution.py runs successfully without import errors.
  • The visualization module and tests both reference the same, well-defined import path.
  • No breaking changes to the public API of plot_numeric_distributions.

貢獻者指南