ELM Annotations Data Duplication about clinical_quality_language HOT 5 CLOSED

I've tried to upload the relevant files but keep running into errors. Please email me if you want them.

from clinical_quality_language.

I've noticed this as well as part of working on getting Annotations to serialize correctly to JSON, so I'm working on this, should have a fix soon.

from clinical_quality_language.

Hi Bryn,
I'm curious if there is a rough ETA on this? My own work is dependent on it, and if its a low priority it might make sense for me to take a stab at it.

from clinical_quality_language.

Apologies for the delay here, I got distracted, trying to get back to this. I've pushed a branch (fix-annotations) that has debugging statements that I've been using to track down the issue, you're welcome to take a look, the more eyes on it the better.

It's clear what's happening, the narrative tracking that "paints" the text to use in the span isn't properly accounting for cases when the parse tree jumps forward and then back (as in a binary infix operator).

That's as far as I got before I got distracted, I'll get back to it as soon as I can, but posting the branch so you can take a look too if you want.

from clinical_quality_language.

This issue was corrected by pull request #94 and the fix will be included in the next release.

from clinical_quality_language.