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

Spring analysis #2

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

Spring analysis #2

wants to merge 71 commits into from

Conversation

EmmaCartuyvels1
Copy link
Collaborator

Anlysing the SPRING data

Copy link
Collaborator

@hansvancalster hansvancalster left a comment

Choose a reason for hiding this comment

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

Ik had problemen om het bestand lokaal te knitten. Dit had met verschillende zaken te maken:

  • het bestand Number of flowers.xlsx is niet mee gepushed
  • er is een mix gebruikt van here::here() + optie root.dir om te verwijzen naar bestanden en een nieuw source.Rproj. Je moet ofwel het ene of het andere gebruiken. Ik ben er vanuit gegaan dat je source.Rproj wil gebruiken en heb hiervoor de nodige aanpassingen gemaakt in PR Fix spring analysis #3. In dat geval verwijs je binnen je Rmd naar relatieve paden en specifieer je root.dir niet.
  • er zat een fout in _verkenning.Rmd waar werd verwezen naar "optional_functional_group" die niet bestaat. Ik heb dit opgelost in PR Fix spring analysis #3 en ben er daar vanuit gegaan dat alle aparte traits kunnen gecombineerd worden in één trait-profiel zodat de venn diagrammen tonen hoeveel van deze unieke trait combinaties geteld werden (per methode of per maand of combi).

Voor de rest ziet het er goed uit. (maar ik zou het stuk over de flowers graag eens lokaal kunnen runnen om te kunnen beoordelen - daar ontbreekt misschien ook nog wat tekstuele uitleg bij de code). Goed dat de checklist issues van de baan zijn 👍

@hansvancalster
Copy link
Collaborator

@EmmaCartuyvels1 kan je nog eens kijken naar de issues van checklist. De checklist action toont dat er nog volgende lintr issues zijn:

--------------------------------------------------------------------------------
5 times "no visible binding for global variable '.data'"
4 times "no visible global function definition for '%>%'"
3 times "no visible global function definition for 'slice_sample'"
2 times "Do not use absolute paths."
1 times "no visible global function definition for 'add_row'"
1 times "no visible global function definition for 'mutate'"
1 times "no visible global function definition for 'tibble'"

Om enkel deze te zien kan je lokaal checklist::check_lintr() uitvoeren.

Copy link
Collaborator

@hansvancalster hansvancalster left a comment

Choose a reason for hiding this comment

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

Ziet er allemaal goed uit 👍

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.

2 participants