feat(radar): lire la réflectivité en mm/h natif (retrait conversion Z-R)#119
Merged
Conversation
Le producteur (infoclimat-pipelines PR #37) sert désormais le champ mm/h natif de Météo-France (WCS REFLECTIVITY_MAX) pour radar_reflectivity (arome_france_convection) et reflectivity_max (arome_france_hd). On retire donc la reconversion Z-R Marshall-Palmer côté client et on lit la valeur directement en mm/h. Le plancher < 0,5 mm/h → NaN est conservé (masque le clair-air et évite les isolignes parasites en mode contours ; le producteur ne plancherise pas). L'échelle mm/h Météociel est inchangée. Noms de variables inchangés → contrat S3 identique.⚠️ À merger seulement une fois le producteur en prod (bucket servant du mm/h), sinon les dBZ bruts seraient interprétés comme des mm/h.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
infoclimat-numeric | 017fe37 | Commit Preview URL Branch Preview URL |
Jul 16 2026, 02:05 PM |
cmer81
marked this pull request as ready for review
July 16, 2026 19:21
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexte
Suite à infoclimat-pipelines#37 (mergé), le producteur bascule la réflectivité radar sur le champ mm/h natif de Météo-France (WCS
REFLECTIVITY_MAXau lieu deREFLECTIVITY_MAX_DBZ). Le client n'a donc plus besoin de reconvertir les dBZ.Changements
om-protocol-settings.ts— suppression de la conversion Z-R Marshall-Palmer dans lepostReadCallback. On lit la valeur directement en mm/h. On conserve le seul plancher< 0,5 mm/h → NaN(masque le clair-air + évite les isolignes parasites en mode contours ; le producteur ne plancherise pas, min 0).radar-reflectivity.ts— échelle mm/h Météociel inchangée ; commentaire mis à jour (donnée désormais native, plus de Z-R).reflectivity_maxen mm/h via le champ natif).Pourquoi c'est mieux que la reconversion (PR #118)
Z=200·R^1,6.Couverture
radar_reflectivity(arome_france_convection) etreflectivity_max(arome_france_hd). Noms de variables inchangés → contrat S3 identique.Vérifications
npm run test→ 463 tests OKnpm run check→ 0 erreurnpm run lint→ prettier + eslint clean🤖 Généré avec Claude Code