CTDC Model now with Versions, correct quotes, and preserved indentations #15
Conversation
Moved references to properties not yet assigned CDEs out of plain sight Continued with model-wide cleaning-up an commenting out Checked and modified key field indicators as necessary Added and/or modified numerous additional CDE references per 06-24-24 input from the SI team
Resolved duplications within SNOMED disease term acceptable values introduced as of the associated SNOMED codes having been commented out
Added CDE references as follows: For 4 properties within the subject node For 2 "date" properties within the targeted_therapy node For 2 "date" properties within the non_ targeted_therapy_node For 2 "date" properties within the radiotherapy node For 3 properties within the specimen node
An update that added CDE versions and kept the single quotes throughout.
There was a problem hiding this comment.
Having done a near line-by-line review of the changes represented here, some of which was done with Todd involved, this looks good to go.
All of the indentation inadvertently removed during the addition of CDE version numbers has been correctly reverted out.
All of the terms that need to be within single quotes are likewise now correctly back within single quotes.
All of the terms that need to quoted within double quotes are likewise now correctly back within double quotes.
And all of the most recent CDE references that I inserted are safely where they should be.
The only issue of note is that there are a handful of properties for which there is a valid CDE reference in place, for which there is no corresponding CDE version number, an issue which Todd is aware of, and planning to address accordingly.
I'm @PhilipMusk, and I approve this PR
No description provided.