maelstrom-research / Rmonize

3 stars 0 forks source link

harmo_process : error with as.Date function in harmo rule + weird output with date format #13

Closed SamEBT closed 1 year ago

SamEBT commented 1 year ago

Test sur R misc avec Rmoniz J'ai une variable au format date avec le format suivant : "2020-05-19 00:00:00 UTC"

Mon objectif etait de la mettre au format : "2020-05-19" avec la harmo_rule suivante as.character(as.Date(collect_date)) Mais erreur lors du harmo_process J'ai egalement essaye as.character(as.Date(collect_date, format = "%Y-%m-%d %H:%M:%S")) idem erreur

j'ai alors tente juste as.character(collect_date) afin de voir si ca venait de la et si l'output donnait quand meme "2020-05-19 00:00:00 UTC". Mais au final en output ca m'a donne : "1588252200" alors que j'avais bien precise le as.character()

J'ai egalement teste uniquement avec as.Date, ca a donne une erreur.

J'ai teste cela suite au fait que sur le serveur CITF (donc version harmonizR assez recente) je n'arrivait pas a obtenir le format "2020-05-19" a partir de "2020-05-19 00:00:00 UTC" en utilisant la harmo_rule as.character(as.Date(collect_date)) En sortie j'avais "4350456-10-29" (donc pas d'erreur avec l'ancienne version mais tout de meme une sortie weird. J'avais teste pas mal de harmo_rule similaire (avec une ecriture plus tidyverse, avec different format etc mais rien n'y faisait j'avais toujours un chiffre a la place d'une date au format text.

GuiFabre commented 1 year ago

to investigate

GuiFabre commented 1 year ago

très beau bug haha !

donc :

Capture d'écran 2023-10-02 101942

Capture d'écran 2023-10-02 102014

Capture d'écran 2023-10-02 102135

Le temps de corriger cela (pour la version à venir, pas pour maintenant), voici ce que tu peux faire;

Remplace ton script par as.Date.POSIXct(collect_date, format = "%Y-%m-%d %H:%M:%S")

Capture d'écran 2023-10-02 102113

Ce qui devrait régler l'affaire

Capture d'écran 2023-10-02 102915

Je ferai la correction plus tard

SamEBT commented 1 year ago

Ok merci pour l'explication et les details !

GuiFabre commented 1 year ago

Actually, sounds working now. @SamEBT can you test it ?

image

GuiFabre commented 1 year ago

Suggestion : Dans ton datashema, tu devrais mettre 'date' au lieu de 'text' pour la variable crée.

image