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

Relecture datatable #324

Merged
merged 13 commits into from
May 27, 2021
Merged

Relecture datatable #324

merged 13 commits into from
May 27, 2021

Conversation

milena-git
Copy link
Contributor

Je remets mes suggestions :)

  • L'affichage .Rmd ne correspond pas toujours aux explications du texte, e.g. affichage des types des variables dans un data.table, ou sa longueur, ou encore, souvent, seules les noms de colonnes s'affichent et pas un aperçu de la table. Je ne sais pas si ça se règle, mais ça peut perturber la lecture (on cherche confirmation de ce qu'on vient de lire dans le chunk, par exemple quand on lit: le data.table indique automatiquement le type des variables (sous le nom de variable), et donne le nombre total d’observations de la table)
  • Sur le passage "il est fortement conseillé de lire la section La fonction d’assignation par référence (ou :=) avant de poursuivre la lecture." Le "avant de poursuivre la lecture" est-il vraiment nécessaire? Sinon, pourquoi ne pas avoir placé carrément la section plus en amont?
  • Une fonctionalité de data.table que j'aime bien est le "roll join"

milena-git and others added 12 commits April 14, 2021 14:24
Montrer plus rapidement un exemple
Redondance de cette ligne avec une suivante.
+ rmq: difficile d'installer doremifasolData (j'ai du aller fouiner dans le DESCRIPTION)
Quelques modifs mineures
Des parenthèses supprimées, une typo, une précision
Co-authored-by: Lino Galiana <33896139+linogaliana@users.noreply.github.com>
Co-authored-by: Olivier Meslin <44379737+oliviermeslin@users.noreply.github.com>
Co-authored-by: Olivier Meslin <44379737+oliviermeslin@users.noreply.github.com>
…passage 'conseil' sur dcast, la description de l'exemple précédent me semblait incohérent avec celui-ci
Copy link
Contributor

@linogaliana linogaliana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

modifs

@linogaliana
Copy link
Contributor

@milena-git je merge et essaie de garder en tête tes commentaires pour des modifs ultérieures (idéalement avant la présentation de cette fiche lors des matinées de découverte). Pour le rollup join, PR welcome car je n'ai jamais utilisé

@linogaliana linogaliana merged commit 1504818 into InseeFrLab:master May 27, 2021
@clerousset
Copy link
Contributor

@linogaliana je veux bien faire la PR rolling join. C'est dans la fiche data.table ou dans les fusions ?

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.

3 participants