Conversation
hansvancalster
left a comment
There was a problem hiding this comment.
Ziet er goed uit 👍 Ik heb een paar comments die je eens moet bekijken. Daarnaast zag ik nog een bestandsnaam adptions to kartering.qmd die je best hernoemt volgens snake_case: adaptations_to_mapping.qmd of zoiets.
| Er werden in totaal 6 verschillende functies geschreven om de observaties te clusteren: | ||
|
|
There was a problem hiding this comment.
Eigenlijk heb je in deze sectie al alle info bijeengebracht die je nodig hebt om roxygen documentatie voor de functies in een R package te schrijven. Dat spaart werk als ooit de vraag zou komen om dit verder uit te werken.
There was a problem hiding this comment.
Ik heb nog nooit een package geschreven.... dus ik weet niet waar je het over hebt ....
There was a problem hiding this comment.
Geen probleem. Het was maar ter info. Ik verwacht niet dat je er iets mee doet. Maar als je er meer over wil weten, zie https://roxygen2.r-lib.org/articles/roxygen2.html
| library(caret) | ||
| library(rprojroot) | ||
| library(devtools) | ||
| devtools::install_github("gbganalyst/forstringr") |
There was a problem hiding this comment.
Ik ken het package niet, maar staat blijkbaar ook op CRAN
hansvancalster
left a comment
There was a problem hiding this comment.
Ik denk dat al mijn commentaren zijn opgelost.
…aantal clustering levels
aanpassing yaml
…en presence functie, toevoegen van past_follow_up.
Aanpassing landgebruik en git structuur