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

Adapt check presence #10

Merged
merged 5 commits into from
Feb 13, 2024
Merged

Conversation

hansvancalster
Copy link
Collaborator

In deze PR een paar verbeteringen aan de check_presence functie zodat je ook als input een character vector van soortnamen kan doorgeven of ze kan lezen vanaf een txt bestand

@slambrechts
Copy link
Collaborator

Ok, bedankt. Lijkt mij ook interessant om rechtstreeks te integreren in de data analyse Rmarkdown? Door eerst de gedetecteerde soorten uit het phyloseq object te halen


species_df <- data.frame(tax_table(physeq)[, "species"], stringsAsFactors = FALSE)

en dan daarop het check presence script lopen?

@hansvancalster
Copy link
Collaborator Author

Ok, bedankt. Lijkt mij ook interessant om rechtstreeks te integreren in de data analyse Rmarkdown? Door eerst de gedetecteerde soorten uit het phyloseq object te halen


species_df <- data.frame(tax_table(physeq)[, "species"], stringsAsFactors = FALSE)

en dan daarop het check presence script lopen?

Goed idee, dat kan je in je PR #8 implementeren. Ik zal nu eerst deze PR mergen en dan Check_occurrence ook in main mergen. Daarna kan je de branch van PR #8 verder werken nadat je git pull origin main hebt uitgevoerd. Je kan deze functie laden in de Rmd via source()

@hansvancalster hansvancalster merged commit 1dc1740 into Check_occurences_GBIF Feb 13, 2024
1 check failed
@hansvancalster hansvancalster deleted the adapt-check-presence branch February 13, 2024 13:08
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