adhit-r/fairmind

Align plot_numeric_distributions module path with tests

Offen

#146 geöffnet am 06.01.2026

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (12 Forks)auto 404
buggood first issuetesting

Repository-Metriken

Stars
 (9 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 2T 6h) (3 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Entscheiden Sie, ob das Modul unter `apps/ml/visualizations/data distribution.py` bleiben oder nach `apps/backend/utils/visualization/data distribution.py` verschoben werden soll, aktualisieren Sie dann den Import in `tests/test distribution.py` entsprechend und führen Sie die Tests aus.
Tech Stack
python
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85