insightsengineering / tern

Table, Listings, and Graphs (TLG) library for common outputs used in clinical trials
https://insightsengineering.github.io/tern/
Other
76 stars 21 forks source link

More general solution to empty string bug coming from `rtables` #598

Closed Melkiades closed 2 years ago

Melkiades commented 2 years ago

This is a more general fix for issue #593 and its scope is to automatically solve this teal.modules.clinical#512.

Melkiades commented 2 years ago

This may be a good start. It is from rtables vignette clinical_trials.Rmd#L197. I tried with the specified splitting function and it does not fix it atm.

Melkiades commented 2 years ago

The empty string "" error was generating bugs only in summarize_logistic, and, therefore, for the moment we keep the fix made in #593. A possible general solution involves a wider standard for missing values that is not always present in real data, which is another matter completely.