search

Merck / metalite

An R package to create metadata structure for ADaM data analysis and reporting
https://merck.github.io/metalite/
GNU General Public License v3.0
15 stars 5 forks source link

`define_parameter()` mandatory arguments #164

Closed yuliasidi closed 4 months ago

yuliasidi commented 4 months ago

Currently define_parameter() requires specification of a lot of arguments that might not be clear to users, e.g., term1, term2. Would you consider simplifying the arguments that are mandatory in the define_parameter()? Also, please note that the current documentation for define_parameter() is very limited and it is difficult for a user who is not familiar with all the details of the metalite packages to figure out how to define a new parameter.

ref: https://github.com/Merck/forestly/issues/20

elong0527 commented 4 months ago

It has been fixed in https://github.com/Merck/forestly/pull/49.

metalite::define_parameter did not require all those arguments, it is being required by the metalite.ae::prepare_ae_specific that is being used by forestly.