-
Notifications
You must be signed in to change notification settings - Fork 18
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
Merge CQL-to-ELM updates into develop-2.0 from the scopes-2.0 branch Integration tests passing [1251 / 1822] #419
Conversation
MinValue or MaxValue => [], // type is a generic type arg | ||
|
||
// special expression types | ||
UnaryWithPrecision uwp => [uwp.operand, uwp.GetPrecision()], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome!
TODO - Check that the CMS measures compile |
|
||
// all functions still take the bundle and context parameters, plus whatver the operands | ||
// to the actual function are. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The logic here is preventing the demo projects from compiling. I'll have to create an issue to revert it back to the original code, and there is already a ticket for the future to try to get this right #397
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll have to revert the InvokeDefinedFunctionThroughRuntimeContext to what is was before #422, then it can be fixed at a later stage. As it is now, the demo projects don't compile
Changes to CqlToElm & ephemeral projects to support translation of 400 HEDIS libraries.