[34] Ajout zone argent #123

Merged

lenaic merged 5 commits from justine/34-zone-argent into develop 2026-09-03 15:21:29 +00:00

Conversation 3 Commits 5 Files changed 33 +3781 -49

justine commented 2026-09-03 13:40:17 +00:00

Member

Copy link

Ce que ça change

Un service services/etl/, sous-paquet silver, qui produit la série exploitable depuis le brut. Sept modules, un par étape, dans l'ordre où elles doivent s'enchaîner : lecture (déplie le bronze, arbitre /current > /readings) → grille (1440 minutes UTC, les créneaux jamais collectés deviennent des lignes nulles) → validation (4 contrôles physiques) → referentiel → imputation (les 3 régimes de l'ADR 0006) → qualite (journal + indicateur) → mesure/job.
Trois tables Parquet dans silver/domain=energy/ : mesure (le schéma §10), journal_capteur (les épisodes ok/absent/rejected par capteur), disponibilite_jour (l'indicateur EF-06, un par site et par jour).

Closes #34

Preuve

Tests

Echantillon office


Echantillon datacenter

Relecture

• Un pair a relu et laissé un commentaire, même court
• Ses remarques sont traitées, ou une réponse explique pourquoi elles ne le sont pas

Ce qui suit le code

• docs/runbooks/ mis à jour, un geste d'exploitation a changé
• docs/adr/ complété, une décision structurante a été prise
• docs/journal.md complété, un incident a été rencontré
• Une nouvelle variable d'environnement est apparue, elle est dans .env.example

Où regarder en priorité

le contenu de /silver

## Ce que ça change Un service services/etl/, sous-paquet silver, qui produit la série exploitable depuis le brut. Sept modules, un par étape, dans l'ordre où elles doivent s'enchaîner : lecture (déplie le bronze, arbitre /current > /readings) → grille (1440 minutes UTC, les créneaux jamais collectés deviennent des lignes nulles) → validation (4 contrôles physiques) → referentiel → imputation (les 3 régimes de l'ADR 0006) → qualite (journal + indicateur) → mesure/job. Trois tables Parquet dans silver/domain=energy/ : mesure (le schéma §10), journal_capteur (les épisodes ok/absent/rejected par capteur), disponibilite_jour (l'indicateur EF-06, un par site et par jour). Closes #34 ## Preuve Tests   Echantillon office   Echantillon datacenter   ## Relecture - [ ] Un pair a relu et laissé un commentaire, même court - [ ] Ses remarques sont traitées, ou une réponse explique pourquoi elles ne le sont pas ## Ce qui suit le code <!-- Ne cocher que ce qui s'applique, supprimer le reste. --> - [ ] `docs/runbooks/` mis à jour, un geste d'exploitation a changé - [x] `docs/adr/` complété, une décision structurante a été prise - [ ] `docs/journal.md` complété, un incident a été rencontré - [ ] Une nouvelle variable d'environnement est apparue, elle est dans `.env.example` ## Où regarder en priorité le contenu de /silver

image.png

70 KiB

image.png

145 KiB

image.png

118 KiB

image.png

115 KiB

image.png

132 KiB

image.png

98 KiB

justine added 1 commit 2026-09-03 13:40:17 +00:00

[34] Ajout zone argent 15f1d7115a

justine requested reviews from marvin, florian 2026-09-03 13:40:26 +00:00

lenaic changed title from [34] Ajout zone argent to WIP: [34] Ajout zone argent 2026-09-03 13:40:57 +00:00

justine added the

Kind/Feature

Kind/Documentation

EC05

Kind/Data

Priority

Critical

labels 2026-09-03 13:48:06 +00:00

justine added this to the EnerVision project 2026-09-03 13:48:28 +00:00

justine self-assigned this 2026-09-03 13:48:30 +00:00

justine added 1 commit 2026-09-03 13:52:55 +00:00

[34] Modification pour qualité Ruff ab3f262c2a

justine added 2 commits 2026-09-03 14:14:03 +00:00

Merge branch 'develop' into justine/34-zone-argent ... 28c59e9b0e

# Conflicts:
#	docs/adr/README.md
#	docs/data/etl-pipeline.md

[34] Merge develop 89ce7df835

justine changed title from WIP: [34] Ajout zone argent to [34] Ajout zone argent 2026-09-03 14:14:16 +00:00

marvin approved these changes 2026-09-03 14:37:35 +00:00

Dismissed

florian approved these changes 2026-09-03 15:15:51 +00:00

Dismissed

justine added 1 commit 2026-09-03 15:16:52 +00:00

Merge branch 'develop' into justine/34-zone-argent e9483f9790

justine dismissed marvin's review 2026-09-03 15:16:52 +00:00

Reason:

New commits pushed, approval review dismissed automatically according to repository settings

justine dismissed florian's review 2026-09-03 15:16:52 +00:00

Reason:

New commits pushed, approval review dismissed automatically according to repository settings

lenaic approved these changes 2026-09-03 15:17:31 +00:00

lenaic left a comment

Copy link

Vérifié plutôt que cru sur parole : tes 160 cas passent, joués en local sur un
PostgreSQL et un MinIO montés pour ça. Chaîne verte, aucun conflit avec develop, et la
fiche 0011 est bien libre — la série va jusqu'à 0010.

Le découpage en sept modules dans l'ordre où les étapes s'enchaînent est ce qui rend le
service lisible : lecture, grille, validation, référentiel, imputation, qualité, mesure.
Et la grille de 1440 minutes qui transforme un créneau jamais collecté en ligne nulle est
le bon choix — c'est ce qui distingue « pas de mesure » de « mesure absente », et la zone
or en dépend.

Un point de coordination, qui ne t'incombe pas. Olivier crée aussi services/etl/
dans sa #124, ouverte en parallèle. Vos deux demandes se marchent dessus sur cinq
fichiers : config.py, __init__.py, pyproject.toml, le README du service et
docs/data/etl-pipeline.md. Ce sont des conflits add/add, donc à résoudre à la main.

Comme tu passes en premier, c'est lui qui rebasera. Je l'en préviens sur sa demande.

Le #34 débloque le modèle et l'inférence, qui attendaient derrière. Bon travail.

Vérifié plutôt que cru sur parole : **tes 160 cas passent**, joués en local sur un PostgreSQL et un MinIO montés pour ça. Chaîne verte, aucun conflit avec `develop`, et la fiche 0011 est bien libre — la série va jusqu'à 0010. Le découpage en sept modules dans l'ordre où les étapes s'enchaînent est ce qui rend le service lisible : lecture, grille, validation, référentiel, imputation, qualité, mesure. Et la grille de 1440 minutes qui transforme un créneau jamais collecté en ligne nulle est le bon choix — c'est ce qui distingue « pas de mesure » de « mesure absente », et la zone or en dépend. **Un point de coordination, qui ne t'incombe pas.** Olivier crée aussi `services/etl/` dans sa #124, ouverte en parallèle. Vos deux demandes se marchent dessus sur cinq fichiers : `config.py`, `__init__.py`, `pyproject.toml`, le README du service et `docs/data/etl-pipeline.md`. Ce sont des conflits add/add, donc à résoudre à la main. Comme tu passes en premier, c'est lui qui rebasera. Je l'en préviens sur sa demande. Le #34 débloque le modèle et l'inférence, qui attendaient derrière. Bon travail.

lenaic referenced this pull request 2026-09-03 15:17:32 +00:00

etl : agrégation et matérialisation de la zone or, export et lignage (#35) #124

lenaic merged commit bcecb81415 into develop 2026-09-03 15:21:29 +00:00

lenaic deleted branch justine/34-zone-argent 2026-09-03 15:21:29 +00:00

lenaic referenced this pull request from a commit 2026-09-03 15:21:30 +00:00

Merge pull request '[34] Ajout zone argent' (#123) from justine/34-zone-argent into develop

gabriel removed this from the EnerVision project 2026-09-03 16:11:14 +00:00

olivier referenced this pull request from a commit 2026-09-04 08:48:32 +00:00

Merge develop : réunit les deux paquets etl sous silver/ et gold/ (#35)

olivier referenced this pull request 2026-09-04 08:51:12 +00:00

etl : agrégation et matérialisation de la zone or, export et lignage (#35) #124

lenaic referenced this pull request 2026-09-07 13:48:55 +00:00

Jalon : develop vers main, la chaîne complète et l'application sur le serveur #170

Sign in to join this conversation.

Reviewers

No reviewers

lenaic

Labels

Clear labels   

Compat/Breaking

Breaking change that won't be backward compatible

  

EC01

Preuve pour l'épreuve Architecture

  

EC02

Preuve pour l'épreuve Management

  

EC03

Preuve pour l'épreuve CI/CD

  

EC04

Preuve pour l'épreuve Cloud

  

EC05

Preuve pour l'épreuve Data et BI

  

EC06

Preuve pour l'épreuve IA

  

Kind/BDD

BDD

  

Kind/Back

Back

  

Kind/Bug

Something is not working

  

Kind/CICD

CICD

  

Kind/Cloud

Cloud

  

Kind/Contenu

Contenu

  

Kind/Data

Data

  

Kind/Documentation

Documentation changes

  

Kind/Enhancement

Improve existing functionality

  

Kind/Feature

New functionality

  

Kind/Front

Front

  

Kind/IA

IA

  

Kind/Infra

Serveur, déploiement, chaîne, supervision

  

Kind/Monitoring

Monitoring

  

Kind/Security

This is security issue

  

Kind/Testing

Issue or pull request related to testing

  

Portée/Post-jury

Périmètre documenté, hors de la fenêtre du 11/09. Repris après le jury. Rien nest supprimé.

  

Priority

Critical

The priority is critical

  

Priority

High

The priority is high

  

Priority

Low

The priority is low

  

Priority

Medium

The priority is medium

  

Reviewed

Confirmed

Issue has been confirmed

  

Reviewed

Duplicate

This issue or pull request already exists

  

Reviewed

Invalid

Invalid issue

  

Reviewed

Won't Fix

This issue won't be fixed

  

Status

Abandoned

Somebody has started to work on this but abandoned work

  

Status

Blocked

Something is blocking this issue or pull request

  

Status

Need More Info

Feedback is required to reproduce issue or to continue work

  

ops/alerte

Alerte d'exploitation (relais Grafana, #42)

No labels

Compat/Breaking

EC01

EC02

EC03

EC04

EC05

EC06

Kind/BDD

Kind/Back

Kind/Bug

Kind/CICD

Kind/Cloud

Kind/Contenu

Kind/Data

Kind/Documentation

Kind/Enhancement

Kind/Feature

Kind/Front

Kind/IA

Kind/Infra

Kind/Monitoring

Kind/Security

Kind/Testing

Portée/Post-jury

Priority

Critical

Priority

High

Priority

Low

Priority

Medium

Reviewed

Confirmed

Reviewed

Duplicate

Reviewed

Invalid

Reviewed

Won't Fix

Status

Abandoned

Status

Blocked

Status

Need More Info

ops/alerte

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

justine

4 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

g2/enervision!123

No description provided.