Skip to content
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

Contexts should have type specifiers #1273

Open
EvanMachusak opened this issue Nov 6, 2023 · 2 comments
Open

Contexts should have type specifiers #1273

EvanMachusak opened this issue Nov 6, 2023 · 2 comments
Labels
cql The issue relates to or requires a change or clarification on the CQL specification

Comments

@EvanMachusak
Copy link

The context definition in a library does not have its TypeSpecifier (presumably a NamedTypeSpecifier) populated.

"contexts": {
      "type": "Library$Contexts",
      "def": [
        {
          "type": "ContextDef",
          "locator": "7:1-7:15",
          "name": "Patient"
        },
        {
          "type": "ContextDef",
          "locator": "12:1-12:18",
          "name": "Unfiltered"
        }
      ]
    }

We need a qualified string here telling us that Patient is {http://hl7.org/fhir}Patient so we can differentiate it from other models that define a type called Patient in use in the same library.

@EvanMachusak EvanMachusak changed the title Contexts should have resultTypeSpecifiers Contexts should have type specifiers Nov 6, 2023
@JPercival JPercival added the cql The issue relates to or requires a change or clarification on the CQL specification label Nov 6, 2023
@JPercival
Copy link
Contributor

@brynrhodes - This requires a change to the ELM spec, possible candidate for CQL 2.0

@ewoutkramer
Copy link

Funny, I was thinking about this just last week as well, wanted to started a discussion, until I realized we are probably just supposed to use Element's resultTypeSpecifier for that. At least, that's what I have done now. Works for us, @EvanMachusak ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cql The issue relates to or requires a change or clarification on the CQL specification
Projects
None yet
Development

No branches or pull requests

3 participants