-
Notifications
You must be signed in to change notification settings - Fork 57
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
Relecture datatable #324
Conversation
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modifs
@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 je veux bien faire la PR rolling join. C'est dans la fiche data.table ou dans les fusions ? |
Je remets mes suggestions :)