Maintenant outre la nouvelle méthode de calcul, les DJU sont calculés avec des seuils de chauffe de 17 et 15 degrés. Nous resterons cette année sur 17 degré. De fait le fichier contient plusieurs feuilles avec dedans des caractères spéciaux (les perluettes).
@JulietteEngelaere comme tu as pu le remarquer suite (on va dire ça n'a pas amélioré le smilblick) à mes modifs, l'intégration continue a planté sur le test de la fonction d'intégration du fichier des DJU.
1 Effectivement, je m'étais dit qu'il aurait fallu que je mette en paramètres les noms des feuilles de calculs à télécharger, c'est sûrement plus propre et en plus ça évite de mettre des caractères spéciaux "le degré". ça peut aussi concourir au plantage ce fameux degré?
2 je me pose la question de laisser à la main du rédacteur la saisie de la période de référence
3 le test d'intégration du fichier, je ne comprends pas comment même avant mes modifs il pouvait fonctionner car le fichier n'existait plus, non?
4 j'avais aussi une question sur le fameux nom du fichier récupéré comment on le récupère ? là j'avoue c'est de la déduction car quand je fais copié le lien à partir du site du SDES j'ai ça comme nom https://www.statistiques.developpement-durable.gouv.fr/media/205/download?inline.
À ta guise, tu peux modifier le script si tu considères que tu perdras moins de temps tant que j'ai mes réponses à mes questions.
1- le degré bien transformé en son caractère ascii ne provoque pas de plantage dans l'absolu, c'est même plus sûr de le mettre transformé en son code ascii dans la fonction plutôt qu'en parametre dans la vignette : la vignette pourrait marcher chez toi sur windows et pas dans l'intégration continue sur linux
2- la période de référence est toujours dans le fichier du SDES, il faut juste changer la case xls
3- Pourquoi l'exemple prep_dju(reg = "52", an = 2022, coeff_dju = 25) fonctionnait l'année dernière avec les url du millésime 2022 de DJU derrière et qu'il ne fonctionne plus maintenant ? ça dépend de comment le SDES met à jour son article de diffusion, s'il cache jusre le lien vers le fichier 2022 au lieu de supprimer le fichier, ça continue de fonctionner un moment, mais là, pas de ils ont l'air d'avoir revu le système de fichier qu'il y a derrière leur site car les url ont changé de structure, et donc ça ne marche plus et c'est tombé sur toi !
4- les url ont dû changer de structure après la diffusion des DJU 2024, l'année dernière on les récupérait simplement avec un clic droit sur le lien de téléchargement
Bref, il y a peut être qch à faire expliciter au SDES en réunion de réseau, peut être que toutes ces modifs dans les url des fichiers qui nous enquiquinent sont en fait une bonne nouvelle et que l'année prochaine, l'url n'aura pas besoin d'être modifiée
Cette fois j'ai bien réinstallé le package avec de Kniter la vignette mais j'ai une erreur :
Error in dplyr::group_by():
! Must group by variables found in .data.
x Column nom_region is not found.
Backtrace:
x
+-bilan.eco.insee::prep_dju(...)
| -... %>% tidyr::unnest(cols = "annee") at Energie/R/prep_dju.R:65:3
la colonne région n'a pas le même nom pour les deux fichiers, je m'étais naïvement dit que tu avais peut être une veille version des ces fichiers, comme on n'avait pas la même url de téléchargement...
Il faut assigner qq à la demande de fusion ici !4 (merged) pour qu'il soit notifié que c'est à son tour d'agir quand on retire le statut de brouillon (draft)
je veux bien mais c'est toi qui a fait les dernières modifications dans ma branche donc à part te dire Ok ça marche, je ne m'étais pas imaginé que tu attendais que je te demande de fusionner.
c'est juste pour que je sois notifiée de ce qui se passe si on attend qqch de ma part, c'est à faire au moment de la création de la demande de fusion, bien avant que j'intervienne pour réparer la CI
je n'avais aucune idée que tu m'attendais