Update limits for R471811 and ESA-metolachlore #184
Conversation
WalkthroughRenommage du key sup_valeur_sanitaire en sup_limite_qualite_sup_3 dans availableCategories.resultats de webapp/lib/polluants.ts, avec mise à jour des labels (“Concentration > 3 µg/L”), des pictos (“warning”) et des couleurs (#d94801, #F3903F). Aucune signature de type ou fonction modifiée. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~7 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
webapp/lib/polluants.ts (1)
470-475: Couleur identique à “sup_limite_qualite” pour R471811: confirmer l’intentionIci,
sup_limite_qualite_sup_3utilise#F3903F, identique àsup_limite_qualite. Pour ESA-métolachlore, vous avez renforcé la couleur (#d94801) pour le seuil > 3 µg/L, ce qui distingue visuellement la gravité. Pour R471811, garder la même couleur risque de rendre la légende moins discriminante (deux niveaux différents, même couleur).
- Si c’est intentionnel (p. ex. décision produit), RAS.
- Sinon, je recommande d’harmoniser la couleur avec
#d94801pour le seuil > 3 µg/L, comme pour ESA-métolachlore.Proposition de patch minimal:
- sup_limite_qualite_sup_3: { - label: "Concentration > 3 µg/L", - couleur: "#F3903F", - couleurFond: "#F3903F", - picto: "warning", - }, + sup_limite_qualite_sup_3: { + label: "Concentration > 3 µg/L", + couleur: "#d94801", + couleurFond: "#d94801", + picto: "warning", + },
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (11)
dbt_/models/intermediate/int__mapping_category_simple.sqlis excluded by none and included by nonedbt_/models/intermediate/pesticide/metabolite_specifique/_int__metabolite_specifique_models.ymlis excluded by none and included by nonedbt_/models/intermediate/pesticide/metabolite_specifique/int__resultats_metabolite_specifique_commune_dernier.sqlis excluded by none and included by nonedbt_/models/intermediate/pesticide/metabolite_specifique/int__resultats_metabolite_specifique_udi_dernier.sqlis excluded by none and included by nonedbt_/models/intermediate/tous/int__resultats_tous_udi_dernier.sqlis excluded by none and included by nonedbt_/seeds/mapping_categories.csvis excluded by!**/*.csvand included by nonedbt_/seeds/references_generations_futures.csvis excluded by!**/*.csvand included by nonedbt_/seeds/schema.ymlis excluded by none and included by nonedbt_/tests/test_metabolite_results.sqlis excluded by none and included by nonedbt_/tests/test_metabolite_specifique_results.sqlis excluded by none and included by nonedbt_/tests/test_pesticide_results.sqlis excluded by none and included by none
📒 Files selected for processing (1)
webapp/lib/polluants.ts(2 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (1)
webapp/lib/polluants.ts (3)
webapp/components/PollutionSidePanel.tsx (1)
PollutionSidePanel(67-226)webapp/components/PollutionMapMarker.tsx (1)
cat(160-175)webapp/components/PollutionMapLegend.tsx (1)
PollutionMapLegend(25-120)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: test
| sup_limite_qualite_sup_3: { | ||
| label: "Concentration > 3 µg/L", | ||
| couleur: "#d94801", | ||
| couleurFond: "#d94801", | ||
| picto: "warning", | ||
| }, |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Renommage + seuil ESA-métolachlore: OK, mais valider l’alignement côté données
Le passage à sup_limite_qualite_sup_3 avec label “Concentration > 3 µg/L”, picto “warning” et couleur plus sévère #d94801 est cohérent et lisible. Point d’attention: si le backend ou la pipeline de classification envoient encore sup_valeur_sanitaire pour cette sous-catégorie, les composants qui mappent resultat -> categoryDetails.resultats[resultValue] risquent de ne plus trouver de correspondance (et donc pas de couleur/picto).
- Confirmer que la valeur émise pour ESA-métolachlore est bien
sup_limite_qualite_sup_3pour > 3 µg/L, et quesup_limite_qualitecouvre toujours 0,9–3 µg/L. - Vérifier qu’aucun code (UI ou data) ne dépend explicitement de l’ancienne clé pour ce polluant.
Proposition de script pour vérifier rapidement les occurrences des clés dans le repo:
🏁 Script executed:
#!/bin/bash
# Liste les occurrences des anciennes/nouvelles clés de résultat
set -euo pipefail
echo "Occurrences de 'sup_valeur_sanitaire' :"
rg -n --no-heading "sup_valeur_sanitaire(?!_)" || true
echo
echo "Occurrences de 'sup_limite_qualite_sup_3' :"
rg -n --no-heading "sup_limite_qualite_sup_3" || trueLength of output: 716
On dirait que rg n’aime pas le lookahead ! Retentons une recherche plus simple pour l’ancienne clé :
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "Occurrences de 'sup_valeur_sanitaire' :"
rg -n --no-heading "sup_valeur_sanitaire" || trueLength of output: 1192
Vérifier et mettre à jour la prise en charge de la nouvelle clé sup_limite_qualite_sup_3
Le script montre que l’ancienne clé sup_valeur_sanitaire est toujours largement utilisée dans la couche UI et dans le mapping des couleurs. Si le backend émet désormais sup_limite_qualite_sup_3 pour ESA-métolachlore > 3 µg/L, ces composants ne trouveront plus de correspondance et risquent d’afficher des états absents ou des erreurs.
Fichiers à adapter :
- webapp/components/PollutionMapLegend.tsx (ligne ~187) : génération de
statNamesur${period}_${category}_sup_valeur_sanitaire - webapp/components/PollutionMapMarker.tsx (ligne ~312) : attente de la clé
"nb_sup_valeur_sanitaire" - webapp/lib/colorMapping.ts (lignes ~74–86) : logique qui teste et colore sur
nb_sup_valeur_sanitaire
Actions recommandées :
- Confirmer avec l’équipe backend que la nouvelle valeur émise est bien
sup_limite_qualite_sup_3et quesup_limite_qualitereste 0,9–3 µg/L. - Étendre ou refactorer le code UI et colorMapping pour gérer également
sup_limite_qualite_sup_3(et son préfixenb_le cas échéant). - Ajouter des tests unitaires/viz manuels pour vérifier l’affichage correct du picto, de la couleur et du libellé « Concentration > 3 µg/L ».
🤖 Prompt for AI Agents
In webapp/lib/polluants.ts around lines 403–408 the new key
sup_limite_qualite_sup_3 is defined but the UI and color-mapping still expect
sup_valeur_sanitaire/nb_sup_valeur_sanitaire; update the UI and mapping to
accept the new key and its prefixed count variant: modify
webapp/components/PollutionMapLegend.tsx (≈line 187) to build statName that
checks both `${period}_${category}_sup_valeur_sanitaire` and
`${period}_${category}_sup_limite_qualite_sup_3`, update
webapp/components/PollutionMapMarker.tsx (≈line 312) to read both
"nb_sup_valeur_sanitaire" and "nb_sup_limite_qualite_sup_3", and change
webapp/lib/colorMapping.ts (≈lines 74–86) to include coloring/label/picto logic
for sup_limite_qualite_sup_3 (and its nb_ prefix) mapping to "Concentration > 3
µg/L"; add unit tests or manual checks to ensure picto, color and label render
correctly; confirm backend key with the team before finalizing.
* update csv file with references * calculations for nitrates are now much simpler (following new specs) * update leng for nitrates * misc * forgot to remove this when cleaning in PR #184
Summary by CodeRabbit