Open bernhard-da opened 2 years ago
after discussions with @ppdewolf this should not yet be merged yet in order to retain consistency between results from sdcMicro / muargus.
Thanks!
Van: bernhard-da @.> Verzonden: vrijdag 22 juli 2022 07:53 Aan: sdcTools/sdcMicro @.> CC: Peter-Paul de Wolf @.>; Mention @.> Onderwerp: [Nieuwsbrief] [Mail werd niet vanaf de juiste machine verstuurd] Re: [sdcTools/sdcMicro] Feature carryalong checks (PR #322)
after discussions with @ppdewolfhttps://github.com/ppdewolf this should not yet be merged yet in order to retain consistency between results from sdcMicro / muargus.
— Reply to this email directly, view it on GitHubhttps://github.com/sdcTools/sdcMicro/pull/322#issuecomment-1192209494, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEWQ4QZCCPGEGTRPAGIWEU3VVIZKRANCNFSM54GQCDZA. You are receiving this because you were mentioned.Message ID: @.**@.>>
this pr adds a check to the recordSwapping library related to
carry_along
variables; it is checked that variables specified ascarry_along
variables are at household-level which means that there is only a single value within each household-id; a corresponding unit-test has been added and additionally a minor issue in utility-functioncreateDat()
that samples test-input data for the record swapping procedure was fixed;