Merck / metalite.ae

An R package for standard adverse events analysis
http://merck.github.io/metalite.ae/
GNU General Public License v3.0
17 stars 4 forks source link

duplicated n_subject function #128

Closed elong0527 closed 1 year ago

elong0527 commented 1 year ago

The n_subject function is an internal function both in metalite and metalite.ae.

Should we export n_subject from metalite and remove n_subject from metalite.ae?

nanxstats commented 1 year ago

I'd say yes? If it's considered universally useful. It's your decision.

elong0527 commented 1 year ago

It will require a newer version of metalite on CRAN, let's hold it for future version :)

fb-elong commented 1 year ago

@wangben718 here is the task related to n_subject