-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Date Error Evaluating DSF with Histories v14 #12
Comments
Hi Jeff, Interesting. Looks like an issue with DateParam construction in the data provider. We are in the process of selecting test patients from the v14 histories. I will be able to debug this once that is finished. However, if you provide the patient id and the measurement period you're using, that may speed up the resolution of this issue (same for issue #13). Thanks, |
I’m seeing this issue with all of the patients so selecting one shouldn’t be difficult.
eJeff
… On Nov 6, 2017, at 4:34 PM, c-schuler ***@***.***> wrote:
Hi Jeff,
Interesting. Looks like an issue with DateParam construction in the data provider. We are in the process of selecting test patients from the v14 histories. I will be able to debug this once that is finished. However, if you provide the patient id and the measurement period you're using, that may speed up the resolution of this issue.
Thanks,
Chris
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#12 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AIO2lDYqaoa8aVv07N1F8oIfpqaGBUznks5sz3tUgaJpZM4QT2TC>.
|
I've written a couple of my own classes but the rest of the computation is entirely cqf-ruler-ruler.
I've also copied two classes from cqf-ruler-ruler and edited them a bit for my purposes:
|
This is now fixed so closing the issue.
|
Measure evaluates fine in-memory with mostly cqf-ruler code but fails on cqf-ruler.
The text was updated successfully, but these errors were encountered: