Spp 003 nl#153
Conversation
corrections
|
That is strange. The slickR package is no longer used. I guess that docker is using an old image?
@karenwuyts The "remove the |
Line 3707 in 1834abd This is due to the out of date branch. I would suggest to rebase this branch to the current main branch. This is something that @hansvancalster should fix. |
janneadolf
left a comment
janneadolf
left a comment
There was a problem hiding this comment.
Here my review of the current protocol version.
I hope my feedback proves useful, feel free to ignore it in case it is not.
Let me know if any of my comments are unclear, require further discussion, or if I misunderstood important parts of the protocol.
| Dit protocol beschrijft het manueel plaatsen van een koppel ondiepe peilbuis en ondiepe piëzometer met automatische druksensor in een geschikte terrestrische en aquatische steekproefeenheid van het meetnet eutrofiëring via het grondwater (meetnet GW_03.3). | ||
| De installatie dient om de stikstoftoestand van het ondiepe freatische grondwater binnen grondwaterafhankelijke habitattypes op te volgen (met behulp va nitraat- en ammoniumconcentratie in het grondwater), maar ook om de verklarende variabelen (vb. stijghoogte, pH, geleidend vermogen) en variabelen voor kwaliteitscontrole (vb. concentratie van de belangrijkste anionen en katione ter controle van de ionenbalans) te bepalen. | ||
|
|
||
| Dit protocol wordt pas doorlopen als de steekproefeenheid recent (minder dan 1,5 jaar voor de installatie) als geschikt werd beoordeeld volgens de locatie-evaluatie (zie protocol [spp-002-nl 2025.03](https://inbo.github.io/protocols/2025.03/index.html) voor terrestrische types). |
There was a problem hiding this comment.
Link not pointing to correct protocol
There was a problem hiding this comment.
Corrected the number in the link. This protocol is currently in a pull request and will be visible on the site once merged.
| - 07_referenties.Rmd | ||
| - 08_appendices.Rmd | ||
| - 09_subprotocols.Rmd | ||
| output_dir: ../../../../../../protocoldocs/protocoldocs/docs/spp/mnm/spp_003_nl_peilbuisinst-mnm03.3 |
There was a problem hiding this comment.
remove redundancy from path and set to recommended location
| output_dir: ../../../../../../protocoldocs/protocoldocs/docs/spp/mnm/spp_003_nl_peilbuisinst-mnm03.3 | |
| output_dir: ../../../../../protocoldocs/docs/spp/mnm/spp_003_nl_peilbuisinst-mnm03.3 |
| Plaats de druksensor aan de draad in de piëzometer. | ||
| Plaats het peilbuisslot in de opening van de buis en draai de vijs op het slot vast. | ||
|
|
||
| ## Geografisch inmeten van peilbuis en piëzometer |
There was a problem hiding this comment.
Suggestion to restructure:
- "geografisch inmeten via RTK-GPS ontvanger, protocol SVP-014"
- "indien niet beschikbaar, dien je dit binnen de 3 maanden in te halen"
|
|
||
| In het geval van aquatische types (lentisch en lotisch), plaats je ook een peilschaal. | ||
| De verklarende variabele voor stikstoftoestand in het grondwater is hier namelijk het verschil tussen grondwaterpeil en oppervlaktewaterpeil. | ||
| Volg hiervoor [protocol SVP-119](https://drive.google.com/file/d/129ch-zbjzQi3BkX0wRm0KSrM6fvwlwH8/view). |
There was a problem hiding this comment.
Should this protocol be mentioned in section "Afhankelijkheden"?
| Volg de stappen in de GIS-applicatie om er zeker van te zijn dat alle nodige informatie digitaal genoteerd wordt. | ||
| Wanneer je tablet of laptop verbonden is met het internet, worden de in de applicatie ingegeven waarden onmiddellijk doorgestuurd naar de server. | ||
| Heeft je tablet of smartphone tijdelijk geen internetconnectie, controleer dan zeker bij het afronden van je veldwerk (vb. wanneer je in de wagen stapt voor je wegrijdt) dat de informatie is doorgestuurd naar de server. | ||
| Controleer voor je op terrein vertrekt steeds of je alle relevante protocollen, veldsleutels en benodigdheden mee hebt. |
There was a problem hiding this comment.
Suggestion to reword (keep more general to avoid confusion, e.g. veldsleutels have not been mentioned, at least not in this protocol); something along the lines:
"Controleer of je alle vermelde benodigheden (sectie X) mee hebt (vb. subprotocollen, materiaal, ...)."
There was a problem hiding this comment.
Here my review of the current protocol version.
I hope my feedback proves useful, feel free to ignore it in case it is not.
Let me know if any of my comments are unclear, require further discussion, or if I misunderstood important parts of the protocol.
Update: I tried submitting the review as an 'approval' to facilitate later merging of this pull request, but received an error message, which I think was Unexpected token '<', "<!DOCTYPE "... is not valid JSON.
Submitting the review as 'comment' (still requiring later approval) worked without throwing an error.
…p-003-nl update following upgrade R4.5 to R4.6
Co-authored-by: Janne Adolf <77746642+janneadolf@users.noreply.github.com>
Co-authored-by: Janne Adolf <77746642+janneadolf@users.noreply.github.com>
10 participants
Description
This project-specific protocol describes the guidelines and steps for the manual installation of ground water wells in terrestrial and aquatic habitat types for monitoring of the ground water chemistry (nitrogen) for the Monitoring Programme for the Natural Environment (MNE; Meetnetten Natuurlijk Milieu, MNM). It makes use of protocol sfp_104_nl.
Related Issue
Task list
Steps by contributor:
protocolhelper::check_frontmatter()and address themprotocolhelper::check_structure()and address themNEWS.RmdReview steps for the author(s):
protocolhelper::check_frontmatter()andprotocolhelper::check_structure()succeeded without errors.To be done by an administrator after review: see guidelines for admins.