adhit-r/fairmind

Align plot_numeric_distributions module path with tests

Aberta

#146 aberto em 6 de jan. de 2026

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)Python (12 forks)auto 404
buggood first issuetesting

Métricas do repositório

Stars
 (9 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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.

Guia do colaborador

Direção de pesquisa
Decida se manter o módulo em `apps/ml/visualizations/data distribution.py` ou movê lo para `apps/backend/utils/visualization/data distribution.py`, depois atualize a importação em `tests/test distribution.py` para corresponder e execute os testes.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85