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

taxonomie en Venn dia gecorrigeerd vr aantal reads per staal #8

Merged
merged 10 commits into from
Feb 21, 2024

Conversation

slambrechts
Copy link
Collaborator

taxonomie plots en Venn diagrammen op basis van data gecorrigeerd voor aantal reads per staal, anders is denk ik bijvoorbeeld gedeelde en unieke OTUs per landgebruik, taxonomie, etc niet vergelijkbaar

…r aantal reads per staal, anders is denk ik bijvoorbeeld gedeelde en unieke OTUs per landgebruik, taxonomie, etc niet vergelijkbaar
… stalen die oorspronkelijk geen metadata hadden zijn toegevoegd
@hansvancalster
Copy link
Collaborator

Dus nu even in je terminal venster zorgen dat branch update_data_analyse actief is, eventuele modified files committen of stashen, en dan git pull origin main en je hebt de functie check_presence.R ook beschikbaar in deze branch.

@slambrechts
Copy link
Collaborator Author

ik heb GBIF check presence toegevoegd, maar het werkt momenteel nog niet. Ligt het aan hoe de functie controleert of het een vector of een input file is, of ligt het aan mijn vector? Ik heb de data in vector formaat gezet, omdat ik wou voorkomen dat ik de data moet wegschrijven als txt en dan opnieuw moet inladen

@hansvancalster
Copy link
Collaborator

Het is een bug in de functie. Zal ik een fix pushen naar deze branch?

@slambrechts
Copy link
Collaborator Author

Ok bedankt. Moet ik dan opnieuw git pull origin main doen nadat check_presence.R aangepast is?

@hansvancalster
Copy link
Collaborator

Ok bedankt. Moet ik dan opnieuw git pull origin main doen nadat check_presence.R aangepast is?

Enkel git pull om de commits die ik gepushed heb lokaal binnen te krijgen

@hansvancalster hansvancalster self-requested a review February 19, 2024 07:32
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.

Dan kan deze branch gemerged worden. Doe jij dat maar @slambrechts hier onderaan op de knop drukken. Lokaal kan je dan de branch main nemen en via git pull de wijzigingen binnenhalen. Daarna kan je de andere PR up to date brengen door die branch te kiezen en git pull origin main uit te voeren en de wijzigingen te pushen.

@hansvancalster
Copy link
Collaborator

Dan kan deze branch gemerged worden. Doe jij dat maar @slambrechts hier onderaan op de knop drukken. Lokaal kan je dan de branch main nemen en via git pull de wijzigingen binnenhalen. Daarna kan je de andere PR up to date brengen door die branch te kiezen en git pull origin main uit te voeren en de wijzigingen te pushen.

ah, je hebt daarvoor de nodige rechten niet. Ik zal het doen

@hansvancalster hansvancalster merged commit 71dc752 into main Feb 21, 2024
1 check failed
@hansvancalster hansvancalster deleted the update_data_analyse_Rmd branch February 21, 2024 14:54
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