logicahealth / vha-kbs-knarts

VHA Knowledge-Based Systems clinical knowledge artifact content development program.
Apache License 2.0
7 stars 3 forks source link

"timing.XXX" should be 'requestedResult.XXX' for 4 lines #71

Open linophth opened 6 years ago

linophth commented 6 years ago

https://github.com/preston/vha-kbs-knarts/blob/3869c083b0e445d3d5646bff1b01af6914ae4df7/content/cardiology_catheterization/o19/CDSK_KRprt_OS_O19PostCardCath.xml#L997

a-dru-desai commented 6 years ago

Linda - are you saying that elsewhere in this KNART or in another KNART B3 used an element code of repetition.lowerBound vs. timing.lowerBound?

linophth commented 6 years ago

No, now that I review it again I understand better what the author was getting at - "timing" refers to the number of tablets given. I retract the particular comment, but I still would be interested in knowing why the number of tablets is modeled as "timing". It does not match what is in the TSR and the TSR makes more sense to me.

catherinestaes commented 6 years ago

line 997, 1000, 1003, 1006 are supposed to be 'requestedResult.XXXX' not 'timing.XXXX' it is currently an error that should be fixed. it needs to match the TSR.

catherinestaes commented 6 years ago

I updated the title of this issue to clarify the problem.

catherinestaes commented 6 years ago

I included 'linopath' to see my finding.

a-dru-desai commented 6 years ago

@kthlnkeating please see Catherine's comment in thread above re: problem to resolve. Thx.

juliaskapik commented 6 years ago

The timing requirements refer to the Technical Note from the CCWP below. There is nothing in that note about repetition.

[Section Selection Behavior: Section all that apply. Optional.] ☐ Aspirin 81 mg tablet orally every day [Technical Note: The clinical provider should be prompted to specify the start times for all of the remaining orders in this subsection.]

The sig for the med is in the order itself and is modeled here:

linophth commented 6 years ago

@catherinestaes I will let Catherine respond to this one.

catherinestaes commented 6 years ago

here is the scoop..... there is an error in the xml where the information from the TSR was transcribed over to the knart. In the TSR, the aspirin information has an element related to 'requestedResult' with the following values: requestedResult.lowerBound=1 requestedResult.upperBound=1 requestedResult.includeLowerBound = TRUE requestedResult.includeUpperBound = TRUE requestedResult.measureSemantic = 421026006 |Oral tablet (qualifier value)|

in the current knart, the text for 'requestedResult' in the first 4 entries above was incorrectly replaced with 'timing'. So, in the current knart it is: timing.lowerBound=1 timing.upperBound=1 timing.includeLowerBound = TRUE timing.includeUpperBound = TRUE requestedResult.measureSemantic = 421026006 |Oral tablet (qualifier value)|

the solution to this problem is to replace the text of 'timing' with 'requestedResult' in the 4 places mentioned above (lines 997, 1000, 1003, 1006). catherine

csp4z commented 6 years ago

Addressed in authoring. Waiting delivery to VA.