search

rivm-syso / KRWQCprotocol

GNU General Public License v3.0
2 stars 0 forks source link

Issue71 #72

Closed gilsvda closed 10 months ago

PatrickvanBeelen commented 1 year ago

Hallo Job,

Leuk dat je me nog op de hoogte houdt. Na 3/4 jaar mis ik mijn werk best nog wel. Mocht je nog een budget hebben dan doe ik graag weer mee.

Met vriendelijke groet,

Patrick

Op ma 17 jul 2023 om 13:59 schreef Job Spijker @.***>:

@.**** approved this pull request.

Zie comment QCvalidate-helpers

In R/QCvalidate-helpers.R https://github.com/rivm-syso/KRWQCprotocol/pull/72#discussion_r1265243460 :

@@ -182,7 +182,7 @@ qcidNietUitvoerbaar <- function(d, d_metingen, benodigdeKolommen){ ungroup() %>% select(qcid, iden)

  • niet_uitvoerbaar <- left_join(niet_uitvoerbaar, id, by = "iden")
  • niet_uitvoerbaar <- left_join(niet_uitvoerbaar, id)

Ik vind het netter om bij een join explicitiet op te geven welke kolommen je voor de join gebruikt. Als je later de code aanpast kom je niet voor verassingen te staan.

— Reply to this email directly, view it on GitHub https://github.com/rivm-syso/KRWQCprotocol/pull/72#pullrequestreview-1532601886, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABHYMOP24EAS7HUB44YFUBDXQUSLBANCNFSM52TEWIPA . You are receiving this because you are subscribed to this thread.Message ID: @.***>