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.

Sign up for GitHub

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

Jump to bottom

Read data pipeline #37

Merged
merged 321 commits into from
Sep 18, 2024
Merged

Read data pipeline #37

merged 321 commits into from
Sep 18, 2024

Conversation

wlangera
Copy link
Collaborator

  • targets pipeline to read and prepare data
  • data validation and pipeline validation
  • markdown file to collect and write sampling frames etc.
  • "dag van de landbouw" figure
  • READMEs for data folders

wlangera and others added 30 commits July 29, 2024 13:58
@wlangera
initial commit
dd054e5
@wlangera
function description
70ccb34
@wlangera
function description
36bfdd2
@wlangera
spatial processing
72615b5
@wlangera
unaltered values
5edc261
@wlangera
cleanup document
bd94863
@wlangera
add conclusion
8d79909
@wlangera
spelling
f6c5317
@wlangera
add some more info
7f6ee19
@wlangera
add col order
4f40a23
@wlangera
taxon matching
87f3180
@wlangera
update checklist
fbe9bc0
@wlangera
spelling
ac98b65
@wlangera
taxon mapping functions
d67f3b0
@wlangera
test taxon mapping
0f593fb
@wlangera
coding style
299312f
@wlangera
cleanup function
0c3ec66
@wlangera
add examples for taxon matching
ff1bc5b
@wlangera
use ellipsis argument
f74c1db
@wlangera
add required packages
49ef9e8
@wlangera
expand example
1e856b4
@wlangera
taxon matching
b553323
@wlangera
conclude taxon automatisation
b955def
@wlangera
Merge pull request #16 from inbo/taxon-mapping
830e388 
Taxon mapping
@wlangera
better explanation
698a5fc
@wlangera
create function to look into vernacular names
37be1ba
@wlangera
clean markdown document
36f604a
@wlangera
filter on extra columns
650cdf5
@wlangera
fix bug in inner join
541f73d
@wlangera
split query in 50
8721402
hansvancalster
hansvancalster reviewed Sep 13, 2024
View reviewed changes
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.

🚀 This is great work of high quality and quantity 👍

I will wait for approval until we have discussed some of my specific comments.

@wlangera
Copy link
Collaborator Author

@hansvancalster I have made adjustments as discussed

hansvancalster
hansvancalster approved these changes Sep 18, 2024
View reviewed changes
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.

Thanks for addressing the review comments. All looks good!

@wlangera wlangera merged commit e38025b into main Sep 18, 2024
1 check passed
@wlangera wlangera deleted the read-data-pipeline branch September 18, 2024 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hansvancalster hansvancalster hansvancalster approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wlangera @hansvancalster