Skip to content

refactor: commune radar aggregated into 5 data-source axes - #36

Merged
ndaen merged 1 commit into
mainfrom
refactor/radar-5-sources
Jul 5, 2026
Merged

refactor: commune radar aggregated into 5 data-source axes#36
ndaen merged 1 commit into
mainfrom
refactor/radar-5-sources

Conversation

@ndaen

@ndaen ndaen commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Aligne le radar de la fiche commune sur le cadre du sujet : 5 axes = les 5 types de données de la problématique, au lieu des 12 dimensions internes.

Ce qui change

  • src/lib/score.ts : SOURCE_GROUPS — mapping des 12 dimensions n_* vers les 5 sources :
    • Immobilier : prix, accès financier
    • Transport : transport, proximité
    • Climat & risques : risques, DPE, ensoleillement
    • Tourisme : tourisme, loisirs
    • Socio-démo : emploi, sécurité, services
  • src/components/score-radar.tsx : chaque axe = moyenne des dimensions renseignées de sa source (NULL ignorés, groupe vide → 0, exposé via sourceProfile testable). Le survol d'un sommet affiche le détail des dimensions du groupe.
  • src/components/commune-panel.tsx : titre « Profil (5 sources de données) ».
  • src/components/score-radar.test.ts : partition exacte des 12 dimensions dans les 5 groupes, moyenne, NULL ignorés, groupe vide.

Vérifié

  • tsc, tests (11/11), build OK.
  • Navigateur : radar 5 axes rendu dans le panneau (?commune=94081), libellés lisibles, valeurs cohérentes (Vitry-sur-Seine forte en Transport).

@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit 8655704):

https://homepedia-webapp-cceb1--pr36-refactor-radar-5-sou-ivhvebak.web.app

(expires Sun, 12 Jul 2026 09:35:23 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d658a29abdbf114e447bbe4bc722d35620c3ead7

@ndaen
ndaen merged commit 582b883 into main Jul 5, 2026
2 checks passed
@ndaen
ndaen deleted the refactor/radar-5-sources branch July 5, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant