Skip to content

feat(contracts): n_obs_rejetees sur les journalières v2/ic/statIC - #12

Merged
citarf merged 1 commit into
mainfrom
citarf/n-obs-rejetees
Aug 4, 2026
Merged

feat(contracts): n_obs_rejetees sur les journalières v2/ic/statIC#12
citarf merged 1 commit into
mainfrom
citarf/n-obs-rejetees

Conversation

@citarf

@citarf citarf commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Colonne ajoutée par le QC des pics de température côté chom-poc-data (infoclimat-labs/chom-poc-data#14, PR infoclimat-labs/chom-poc-data#47), déjà déployée en production.

Ce que la colonne décrit

n_obs_rejetees (BIGINT) compte, pour chaque jour-station, les observations de température écartées par le contrôle qualité des pics isolés. Elle vaut 0 dans l'écrasante majorité des lignes.

Ajoutée aux trois datasets journaliere : gold-climato-v2, gold-ic, gold-statIC.

Pourquoi elle appartient au contrat

rowSchema étant en additionalProperties: true, la colonne sort correctement sans cette PR — mais non documentée. Le schéma d'un dataset est précisément ce qu'un contrat ODCS décrit ; une colonne nouvelle y a sa place.

Ce n'est pas la frontière posée par infoclimat-labs/chom-poc-data#9, qui écartait de retoucher des descriptions déjà correctes sur le fond. Ici on ajoute une colonne.

Ampleur du correctif

Table Jours corrigés Stations Obs écartées
gold_v2.journaliere 156 965 13 919 163 882
gold_ic.journaliere 156 820 13 860 163 717
gold_statIC.journaliere 900 207 1 246

Ce qui n'est pas touché

taux_qc_valide de gold-statIC reste inchangé : il mesure la part d'observations dans les bornes physiques [-90, +60] °C, contrôle distinct de celui-ci. Les deux cohabitent sans redondance.

La liste nominative des observations écartées est publiée sur /qc/pic_temperature, avec la profondeur, le seuil et la marge appliqués.

🤖 Generated with Claude Code

Colonne ajoutée par le QC des pics isolés (infoclimat-labs/chom-poc-data#14).
Le schéma d'un dataset est ce qu'un contrat ODCS décrit : une colonne
nouvelle y appartient, contrairement aux descriptions déjà correctes que #9
avait délibérément laissées en place.

taux_qc_valide (gold-statIC) n'est pas touché : il mesure la part dans les
bornes physiques, contrôle distinct de celui-ci.
@citarf
citarf merged commit da6e74f into main Aug 4, 2026
3 checks passed
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