brittanyblouin / ANCRTAdjust

An R package to adjust routine HIV testing data from antenatal care to reduce bias in estimating HIV prevalence trends
MIT License
2 stars 3 forks source link

IPCW #11

Open ellessenne opened 4 years ago

ellessenne commented 4 years ago

Please include more details on the weighting procedure (e.g. mathematical formulae and so on) in the documentation.

Also, can the procedure handle intermittent missingness or is it assuming "censoring" in the sense that once unobserved, it will always be unobserved from there onwards?

seabbs commented 4 years ago

Agreed - this feels far too much like a black box to me.

It would be nice to have a vignette on this - potentially with an example comparing using and not using (and/or some comment on other methods).

For this #openjournals/joss-reviews/issues/1740 review

m-maheu-giroux commented 4 years ago

Fair point. The mathematical formula has now been included in the package documentation and it should now be clear how IPCW was implemented.

ellessenne commented 4 years ago

Thanks, but I still think you may want to include more details - how do you estimate the probability of censoring?

I would actually suggest adding a (short) vignette that describes the relevant methodology, I believe it is super important to not have any doubt regarding what the package does.