cdisc-org / DDF-RA

This is the repository for all code and documentation for the DDF-RA project.
MIT License
17 stars 1 forks source link

Fix some CT inconsistencies #417

Closed drewcdisc closed 1 month ago

drewcdisc commented 1 month ago

CT Fixes to make to existing:

  1. Activity.label: Update the CDISC Submission value and Definition
  2. Recode Class AnalysisPopulation: C188854 to CNEW
  3. Update submission value for AnalysisPopulation.description:
  4. Update submission value for Endpoint.name and Endpoint.label

Plus any additional ones I find

BSnoeijerCD commented 1 month ago

@EMuhlbradt @czwickl : while updating the CPT mapping I found another inconsistency that you can take care of: StudyDesign trialIntentTypes indicates that it uses codelist C66736 - Trial Intent Type Response, while the codelist URL refers to C124307: Treatment Intent. I believe the first one is correct. Furthermore, I wonder whether trial Intent type should not be singular instead of plural. The mapping to CPT indicates only one to be filled.

EMuhlbradt commented 1 month ago

@BSnoeijerCD @dih-cdisc @czwickl see the attached for the full list of changes DDF_417.xlsx

dih-cdisc commented 1 month ago

Furthermore, I wonder whether trial Intent type should not be singular instead of plural. The mapping to CPT indicates only one to be filled.

As we discussed yesterday, SDTM TS domain allows for multiple values, hence why USDM allows multiple. Note in IG?

BSnoeijerCD commented 1 month ago

Furthermore, I wonder whether trial Intent type should not be singular instead of plural. The mapping to CPT indicates only one to be filled.

As we discussed yesterday, SDTM TS domain allows for multiple values, hence why USDM allows multiple. Note in IG?

Yes. Added this information to IG - CPT mapping of the CPT Primary Purpose variable.

BSnoeijerCD commented 1 month ago

@BSnoeijerCD @dih-cdisc @czwickl see the attached for the full list of changes DDF_417.xlsx

In general/summary: if all other items have Study added then adding that would indeed make it indeed more consistent. Otherwise, I would leave it as is since it is not included in class and or relationship names. I would not add clinical to the proposed changes of preferred terms as this can make it too restrictive and confusing.

dih-cdisc commented 1 month ago

@BSnoeijerCD @dih-cdisc @czwickl see the attached for the full list of changes DDF_417.xlsx

  • Not sure why activity should be changed to Study Activity as class name and relationship to class does not include Study. Also Preferred term change to "Clinical Study Activity Label" might be seen as too restrictive as it might be other activities then clinical activities that are presented in the Schedule of activities.
  • Same for endpoints -> not including study there.
  • For element - design is included now while this is not the case for endpoints for example. Not sure why it is needed.
  • Encounters can be different from clinical encounters -> Can be an entry of a PRO or call to check whether drugs are taken for example. So I think clinical makes it too narrow.
  • Same for eligibility Criteria => not sure why study needs to be added.

In general/summary: if all other items have Study added then adding that would indeed make it indeed more consistent. Otherwise, I would leave it as is since it is not included in class and or relationship names. I would not add clinical to the proposed changes of preferred terms as this can make it too restrictive and confusing.

@BSnoeijerCD look at the changes in the context of the whole CT, e.g. for Activity, not the label preferred term is "different" in style to that of name and description.

Screenshot 2024-07-08 at 11 13 45

BSnoeijerCD commented 1 month ago

OK. @EMuhlbradt @czwickl Then is it's good to be consistent and align with the preferred name for the class itself. Also be cautious with using clinical in the label as it might be confusing.

Activity =>use: Activity or Study Activity (Study Activity is now the class label) AnalysisPopulation => Target Study Population for Analysis is now the class label. So I would adjust that one as well to "Analysis Population" then. StudyEndpoint => Proposed changes are making it consistent. So OK. StudyElement => Proposed change makes it consistent. So OK. Encounter => Can we remove clinical everywhere instead of adding it to label as that might be confusing because the encounter does not have to be a clinical one or does not have to be at the clinic? EligibilityCriterion => Proposed change makes it consistent. So OK.

BSnoeijerCD commented 1 month ago

@EMuhlbradt @czwickl

EMuhlbradt commented 1 month ago

@BSnoeijerCD @dih-cdisc Here is the updated list of changes to go along with this ticket. 417_2024-07-10.xlsx

BSnoeijerCD commented 1 month ago

@EMuhlbradt @czwickl : thank you. Looks good. a few small issues:

For the rest it is ok

EMuhlbradt commented 1 month ago

@BSnoeijerCD @dih-cdisc QA issues above fixed and pushed to Github deliverables in 3-4-0 release.

BSnoeijerCD commented 1 month ago

@EMuhlbradt @czwickl

EMuhlbradt commented 1 month ago

@BSnoeijerCD done and pushed to 3-4-0 release on Github