Skip to content

fix: harmoniser l’affichage brut/filtré en µV - #278

Merged
raveriss merged 1 commit into
mainfrom
codex/verifier-l-unite-des-valeurs-en-ordonnee
Feb 5, 2026
Merged

fix: harmoniser l’affichage brut/filtré en µV#278
raveriss merged 1 commit into
mainfrom
codex/verifier-l-unite-des-valeurs-en-ordonnee

Conversation

@raveriss

@raveriss raveriss commented Feb 5, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Rendre l’échelle des plots cohérente entre signal brut et signal filtré en affichant explicitement les amplitudes en microvolts pour éviter la confusion due aux valeurs en volts.
  • Clarifier les labels d’axes pour que les figures soient immédiatement compréhensibles lors de la revue/défense (WBS 3.3 — visualisation brut vs filtré).

Description

  • Ajout d’une constante VOLTS_TO_MICROVOLTS = 1e6 et utilisation centralisée de cette constante dans scripts/visualize_raw_filtered.py pour convertir les données affichées de volts en µV.
  • Conversion appliquée à raw.get_data() et filtered.get_data() avant tracé pour garantir une unité homogène sur les deux axes.
  • Mise à jour des labels Y en "Amplitude (µV)" et "Amplitude filtrée (µV)" dans scripts/visualize_raw_filtered.py.
  • Tests ajustés dans tests/test_visualize_raw_filtered.py pour valider la conversion (attendus multipliés par 1e6) et les nouveaux labels.

Testing

  • Exécution ciblée : poetry run pytest tests/test_visualize_raw_filtered.py -q — OK (20 passed).
  • Suite complète : poetry run pytest -q — OK (577 passed, 22 warnings).
  • Pre-commit et static analysis : poetry run pre-commit run --all-files, poetry run black --check ., poetry run isort --check-only ., poetry run ruff check ., poetry run mypy src scripts tests — OK (all passed).
  • Audit & couverture : poetry run pip-audit --progress-spinner=off (initial pip version warning fixed by upgrading pip then re-run — final audit OK/known-skip for local package), make cov — OK (coverage report produced; total coverage 90% as reported).

Codex Task

@codecov

codecov Bot commented Feb 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@raveriss
raveriss marked this pull request as ready for review February 5, 2026 14:27
@raveriss
raveriss merged commit b9a7a8f into main Feb 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant