search

beckerbenj / eatGADS

https://beckerbenj.github.io/eatGADS/
1 stars 1 forks source link

Refactor recode_labels() #59

Open beckerbenj opened 1 year ago

beckerbenj commented 1 year ago

Currently, recode_labels() is not very readable and probably more complex than it should be.

It should be also adressed that errors and warnings should be meaningful for both applyChangeMeta() as well as recodeGADS(). See, for instance,

stop("Duplicated values in 'value_new' causing conflicting meta data in variable ", var_name, ": ",
             paste(dup_recode_values, collapse = ", "), ". Use existingMeta = 'drop' or 'ignore' to drop all related meta data.")
beckerbenj commented 2 weeks ago

Maybe also rename the options for existingMeta (because no one - including myelf - understands them)....