Skip to content

Sfp 408 nl veldprotocol LSVI stilstaande wateren - #142

Merged
hansvancalster merged 33 commits into
mainfrom
sfp-408-nl
Nov 21, 2025
Merged

Sfp 408 nl veldprotocol LSVI stilstaande wateren#142
hansvancalster merged 33 commits into
mainfrom
sfp-408-nl

Conversation

@Florian9041

@Florian9041 Florian9041 commented Oct 16, 2025

Copy link
Copy Markdown
Contributor

Description

Veldprotocol LSVI stilstaande wateren 31xx en 2190a

Related Issue

Task list

Steps by contributor:

  • Add description to this pull request (under "## Description")
  • Open the dropdown (triangle) near 'create pull request' and choose 'Create draft pull request'
  • Check for potential problems by running protocolhelper::check_frontmatter() and address them
  • Check for potential problems by running protocolhelper::check_structure() and address them
  • Add further commits if needed and push them to GitHub
  • Update the protocol-specific NEWS.Rmd
  • Mark the pull request as 'ready for review'

Review steps for the author(s):

  • Add reviewers, at least one subject-matter specialist and one administrator
  • Wait for review comments and address them
  • Iterate until reviewer approvals (merging the pull request will be done by an administrator)
  • Verify that the checks done by continuous integration succeeded. These will check if protocolhelper::check_frontmatter() and protocolhelper::check_structure() succeeded without errors.

To be done by an administrator after review: see guidelines for admins.

@hansvancalster

Copy link
Copy Markdown
Collaborator

@Florian9041 kan je aanduiden bij "steps by contributor" (cf je eerste bericht in deze conversation) welke stappen je al gedaan hebt?

@Florian9041

Florian9041 commented Oct 20, 2025

Copy link
Copy Markdown
Contributor Author

Steps by contributor:

  • Check for potential problems by running protocolhelper::check_frontmatter() and address them

@hansvancalster Dit geeft de volgende foutmelding (bij check_frontmatter("sfp-408-nl"):
Error in libgit2::git_checkout_tree :
failed rmdir - 'C:/Users/florian_vanhecke/Documents/Projecten/protocolsource/source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/' is locked: Toegang geweigerd.

@hansvancalster

Copy link
Copy Markdown
Collaborator

Steps by contributor:

  • Check for potential problems by running protocolhelper::check_frontmatter() and address them

@hansvancalster Dit geeft de volgende foutmelding (bij check_frontmatter("sfp-408-nl"): Error in libgit2::git_checkout_tree : failed rmdir - 'C:/Users/florian_vanhecke/Documents/Projecten/protocolsource/source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/' is locked: Toegang geweigerd.

Dat is een lokaal probleem dat te maken heeft met schrijfrechten, maar je mag er vanuit gaan dat de checks allemaal slagen want ze worden ook hier in deze PR getest en daar slagen ze (zie het groene vinkje bij "check the protocol on pull request").

@Florian9041 Florian9041 reopened this Oct 20, 2025
@Florian9041

Florian9041 commented Oct 21, 2025

Copy link
Copy Markdown
Contributor Author

@hansvancalster klopt het dat het nu als "ready to review" staat?
Ik weet niet hoe ik de status moet veranderen

@hansvancalster

Copy link
Copy Markdown
Collaborator

@hansvancalster klopt het dat het nu als "ready to review" staat? Ik weet niet hoe ik de status moet veranderen

Ja klopt, de status kan je veranderen bij "convert to draft"

Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/NEWS.md Outdated
@Florian9041 Florian9041 self-assigned this Oct 21, 2025

@anleyssen anleyssen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Het ziet er al goed uit. Ik heb nog enkele suggesties voor aanpassingen via comments toegevoegd.

Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/.zenodo.json Outdated
Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/03_beperkingen.Rmd Outdated
Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/07_werkwijze.Rmd Outdated
Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/12_appendices.Rmd Outdated
Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/12_appendices.Rmd
Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/12_appendices.Rmd
Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/12_appendices.Rmd
Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/12_appendices.Rmd Outdated
Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/index.Rmd Outdated
subtitel verwijderd in index
toevoeging materiaal bootwerk, correctie aparatuur naar apparatuur
aanpassing spaties in referenties
@hansvancalster

Copy link
Copy Markdown
Collaborator

@Florian9041 als je klaar bent met de requested changes van An, moet je haar nog vragen om dit na te kijken en te approven. Ik zag ook nog een error van check_frontmatter: je moet de subtitle lijn verwijderen uit index.Rmd als er geen subtitle is. Je kan An om een nieuwe review vragen bovenaan rechts door op de re-request review knop naast haar naam te klikken

@Florian9041

Copy link
Copy Markdown
Contributor Author

@anleyssen ik denk dat nu alles in orde is. Ik heb opnieuw een review door jou aangevraagd, wanneer dit voor jou goed is kan ik ook aan @hansvancalster een 2de review vragen om alles af te ronden.

@Florian9041

Florian9041 commented Nov 17, 2025 via email

Copy link
Copy Markdown
Contributor Author

@anleyssen anleyssen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Op twee kleinigheidjes na, is het in orde voor mij

Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/06_benodigdheden.Rmd Outdated
Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/07_werkwijze.Rmd Outdated
####Bedekking individuele soorten --> #### Bedekking individuele soorten
Verwijder underscore checklist_boot voor correct renderen van verwijzing
@Florian9041

Copy link
Copy Markdown
Contributor Author

@hansvancalster dan heb ik alleen nog jouw goedkeuring nodig denk ik voor de publicatie.

Comment thread source/sfp/4_vegetation/sfp_408_nl_lsvi_stilwater/02_onderwerp.Rmd
@Florian9041

Copy link
Copy Markdown
Contributor Author

@hansvancalster is in orde nu

@hansvancalster

Copy link
Copy Markdown
Collaborator

@hansvancalster is in orde nu

Ik vermoed dat de tekst nu twee keer verschijnt in de pdf / html. Ik denk dat het enkel tussen die description start end moet geplaatst worden. Heb je dat gecheckt?

@hansvancalster

Copy link
Copy Markdown
Collaborator

@hansvancalster is in orde nu

Nog vergeten te zeggen dat je ook nog checklist::update_citation() moet uitvoeren om te zorgen dat die description ook in .zenodo.json wordt toegevoegd. (cf protocolhelper::check_all() die dit aangeeft)

Het doel van het veldprotocol...

... De resultaten van dit meetnet.
@Florian9041

Florian9041 commented Nov 20, 2025 via email

Copy link
Copy Markdown
Contributor Author

@hansvancalster
hansvancalster merged commit cf7652f into main Nov 21, 2025
3 checks passed
@hansvancalster
hansvancalster deleted the sfp-408-nl branch November 21, 2025 08:13
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.

3 participants