insightsengineering / teal.modules.clinical

Provides teal modules for the standard clinical trials outputs
https://insightsengineering.github.io/teal.modules.clinical/
Other
32 stars 17 forks source link

(safety sample app) the SMQXXNAM/CQXXNAM basket that does nothing #162

Closed kpagacz closed 2 years ago

kpagacz commented 2 years ago

Selecting/deselecting the value does not change the numbers in the tables. I don't really know what exactly this is supposed to change, so this might be perfectly fine behavour.

imazubi commented 2 years ago

Hi @kpagacz , yeah this is a correct behaviour. There is no adverse event that fall in this basket.

adae <- synthetic_cdisc_data("latest")$adae
> all(is.na(adae$SMQ02NAM))
[1] TRUE

We can close this issue.

anajens commented 2 years ago

I think maybe there are some small improvements we can make to inform users about this. I will look into a proposal.

anajens commented 2 years ago

This depends on https://github.com/insightsengineering/tern/issues/195. Afterwards the template function can be adjusted to introduce the rows with 0s.

anajens commented 2 years ago

PR in tern is ready here: https://github.com/insightsengineering/tern/pull/202 so unblocking this issue.