-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Erreurs dans le parseur Enedis à cause du changement d'heure #82
Comments
Je vais regarder ça, merci pour le retour. |
Bon j'aurai pas le temps de tester mais le rapport Enedis donne bien le fuseau contrairement à celui d'EDF. Par contre, il faut que je vérifie en sortie si on a pas une vérification sur un modulo du pas. Cela poserait problème si on se retrouve avec 50 pas au lieu de 48 par exemple. |
Je confirme, on fait un modulo plus tard, la valeur ne change pas pour les 3 valeurs ajoutées en pas de 15. Du coup, le calcul pour la journée est encore plus éloigné de la valeur réelle. Faute de mieux, je vais laisser cette approximation pour 2 jours dans l'année, mais je suis preneur de retours. |
Je n'ai pas compris
Attention, il y a deux problèmes distinct. 1er problème : les doublonsCe n'est pas un problème d'avoir 50 entrées au lieu de 48 puisque le modulo donne toujours le même chiffre. Par contre en y regardant de près, c'est pire pour le changement d'heure du printemps. Dans ce cas, il y a 46 entrées au lieu de 48, ce qui a pour effet que 2eme problème : le 'bug' de ISOStringil me semble que la solution que j'ai proposée permette de résoudre le problème. |
Dans enedisParser.js, ligne 36, il écrit qu'il n'y a jamais de doublon chez ENEDIS. C'est inexact, cela se produit lors du changement d'heure d'hiver. Par exemple, dans mes données :
2023-10-29T01:00:00+02:00;120
2023-10-29T01:30:00+02:00;154
2023-10-29T02:00:00+02:00;126
2023-10-29T02:30:00+02:00;128
2023-10-29T02:00:00+01:00;146
2023-10-29T02:30:00+01:00;94
2023-10-29T03:00:00+01:00;164
2023-10-29T03:30:00+01:00;116
Ce qui a pour effet d'ignorer la consommation de deux mesures correspondant à l'heure du changement d'heure
The text was updated successfully, but these errors were encountered: