includeLowerBound should be includeUpperBound and boundary values are missing

logicahealth / vha-kbs-knarts

VHA Knowledge-Based Systems clinical knowledge artifact content development program.

Apache License 2.0

7 stars 3 forks source link

includeLowerBound should be includeUpperBound and boundary values are missing #224

Closed scottdwood closed 6 years ago

scottdwood commented 6 years ago

https://github.com/preston/vha-kbs-knarts/blob/f789d7c01af822044e729ed13fc8cf0e5284292e/content/neurosurgery_cervical_radiculopathy_clinical_white_paper/b39/CDSK_KRprt_CRDT_B39CervRadi.xml#L1274

Copy/paste error of field name in TSR and carried over into XML. Also, the actual values for those boundaries are missing (timing.lowerBound, timing.upperBound).

amsimms commented 6 years ago

field name in the original TSR included a duplicate of timing.includeLowerBound. This has been fixed in the TSR template, and fixed in the KNART during rework. Missing lowerBound and upperBound properties will also added.

preston commented 6 years ago

@amsimms Thanks! Preemptively removing from the technical review queue.