-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update code #42
Update code #42
Conversation
… aangeroepen functie curvekarakteristieken())
bij deze heb ik dus de problemen i.v.m. het dynamisch validatierapport van issue #32 opgelost |
Bij deze is ook het issue over functie initiatie (#34) opgelost |
…validatie.afgeleid() en validatie.lokaal() (issue #36)
@leymanan Hiermee heb ik de issues opgelost die jij aangebracht had. Terwijl je de oplossingen nakijkt en uittest (geen haast bij, liever dat je het grondig doet), zal ik issue #30 nog in orde brengen in deze PR. Daarna zal ik een review request sturen (maar geef gerust verder in de issues aan als iets nog niet volledig in orde is, dat werkt wel handig). En dan stel ik voor om nadien in een aparte PR nog eventjes de afwerking van het package te doen (website met documentatie, versienummer en alvast 'news'-pagina aanmaken om bij nieuwe versies de aanpassingen toe te voegen, dhcurve toevoegen aan deze website,...). |
…() en validatie.lokaal() (issue #36)
Ik beschouw dit als afgewerkt, ben ik iets vergeten? |
Is ook volgens mij afgewerkt! Bedankt!
Ga jij mergen?
Grts, anja
Op di 15 dec. 2020 om 17:20 schreef ElsLommelen <notifications@github.com>:
… Ik beschouw dit als afgewerkt, ben ik iets vergeten?
(Issue #30 <#30> vind ik minder
dringend omdat het geen impact heeft op de gebruikers, ik stel voor om dit
uit te stellen voor een volgende versie.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#42 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AGKXN5HBUVUCXWPFRGP3OZLSU6EEJANCNFSM4TFMFUJQ>
.
--
*Anja Leyman *
Expert Cel Beheerplanning en Monitoring
--------------------
*Ik werk tijdelijk niet op woensdag- en vrijdagnamiddag.*
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
Vlaamse overheid
AGENTSCHAP *NATUUR & BOS*
Standplaats Instituut voor Natuur- en Bosonderzoek (INBO)
Gaverstraat 4, 9500 Geraardsbergen
T: 054 436 182 M: 0495 14 90 60
E-mail: *anja.leyman@vlaanderen.be <anja.leyman@lne.vlaanderen.be>*
*www.natuurenbos.be <http://www.natuurenbos.be/>*
---
De inhoud van dit bericht en eventuele bijlage(n) verbinden het Agentschap
voor Natuur en Bos niet, zolang niet bevestigd door een geldig ondertekend
document
|
Ok, ik zal mergen, en dan meteen de andere PR verder in orde brengen. |
jij bedankt!!
Op wo 16 dec. 2020 om 10:12 schreef ElsLommelen <notifications@github.com>:
… Ok, ik zal mergen, en dan meteen de andere PR verder in orde brengen.
Bedankt voor je hulp!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#42 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AGKXN5G6XX3YKI6EFPPKVT3SVB2ZFANCNFSM4TFMFUJQ>
.
--
*Anja Leyman *
Expert Cel Beheerplanning en Monitoring
--------------------
*Ik werk tijdelijk niet op woensdag- en vrijdagnamiddag.*
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
Vlaamse overheid
AGENTSCHAP *NATUUR & BOS*
Standplaats Instituut voor Natuur- en Bosonderzoek (INBO)
Gaverstraat 4, 9500 Geraardsbergen
T: 054 436 182 M: 0495 14 90 60
E-mail: *anja.leyman@vlaanderen.be <anja.leyman@lne.vlaanderen.be>*
*www.natuurenbos.be <http://www.natuurenbos.be/>*
---
De inhoud van dit bericht en eventuele bijlage(n) verbinden het Agentschap
voor Natuur en Bos niet, zolang niet bevestigd door een geldig ondertekend
document
|
Aanpassing van de verouderde code aan recente aanpassingen in packages (bv. nieuwe semantiek voor dplyr) voor zover dit nodig is voor de goede werking van het package.