insightsengineering/random.cdisc.data

ADAE: Incorrect Variable Name

Closed this issue · 3 comments

npaszty commented

What happened?

believe that ADTHAUT should be AEDTHAUT

found variable in scda.2021 but don't see it in the yaml definition file or in the radae function.

TODO

Replace ADTHAUT to AEDTHAUT

sessionInfo()

don't think this is relevant information for this issue.

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct.

Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines.

Security Policy

  • I agree to follow this project's Security Policy.
edelarua commented

@npaszty the ADTHAUT variable is in the ADSL dataset in random.cdisc.data and is only inherited from there by ADAE (this is also why it is documented in the ADSL.yml file). According to GDSR when the variable is in ADSL it is named ADTHAUT and when it is in ADAE it is AEDTHAUT. In light of this, is this change still needed (we could make a new AEDTHAUT variable in ADAE if so)?

shajoezhu commented

rename ADTHAUT to AEDTHAUT in adae? what do you think @npaszty

npaszty commented

@shajoezhu @edelarua
right! didn't think to look upstream in ADSL. I guess in this case under the RCD context we just leave it alone.