Skip to content
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

Ajout fichiers plats #7

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

ddotta
Copy link

@ddotta ddotta commented Apr 11, 2022

PR en lien avec cette issue d'utilitr sur la reproductibilité des imports de fichiers.

Avec l'idée de @py-b de rajouter des données aux formats plats et tableurs directement dans {doremifasolData}.

Dans le détail, cette PR contient :

  • L'ajout des librairies manquantes dans les scripts R du dossier data-raw/
  • L'ajout du fichier csv commune-01012019.csv dans inst/extdata avec le script qui permet de le générer data-raw/cog_com_2019_csv.R
  • L'ajout de la fonction doremifasolData_exemple() qui permet de lire facilement les fichiers qui seront contenus dans inst/extdata. J'ai hésité à ajouter cette fonction dans {doremifasol} plutôt que {doremifasolData} mais je me dis que ce serait mieux que {doremifasolData} soit indépendant et contienne la fonction pour les utilisations extérieures (comme utilitr).

L'idée est de pouvoir remplacer les exemples d'utilitr avec les fichiers plats ou aux formats tableurs de {doremifasolData}.
Ainsi ce chunk pourrait être remplacé par :

library(readr)
library(doremifasolData)
communes <- read_csv(doremifasolData_exemple("communes-01012019.csv"),
                     locale = locale(encoding ="UTF-8"),
                     col_types = cols(com = col_character(),
                                      reg = col_integer())
                     )[,c("com","reg")]
names(communes)

Si vous êtes ok sur le principe de la PR @py-b, @pierre-lamarche @linogaliana @oliviermeslin, il resterait à :

  • Ajouter les autres fichiers plats et tableurs
  • Compléter le pkgdown du package
  • Et remplacer les exemples des fichies d'import d'utilitr

@ddotta ddotta changed the title Ajout extdata Ajout fichiers plats et tableurs Apr 11, 2022
@ddotta
Copy link
Author

ddotta commented Apr 12, 2022

Finalement pour les besoins d'utilitr, ajouter seulement le fichier communes-01012019.csv suffit.
Pour les fichiers tableurs, ils sont sans lien avec insee.fr et donc n'ont rien à faire dans doremifasolData à mon avis
J'ai donc simplement ajouté de la doc dans le site pkgdown

@ddotta ddotta changed the title Ajout fichiers plats et tableurs Ajout fichiers plats Apr 12, 2022
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