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

Add support for multiple valuesets and system filters to the ValueSetExpander #2555

Closed
ewoutkramer opened this issue Aug 1, 2023 · 1 comment · Fixed by #2558
Closed
Assignees

Comments

@ewoutkramer
Copy link
Member

The CQL engine uses valuesets, and some of the valuesets in its test set have multiple compose.include.valueset. We do not support that yet. In order to get the SDK's terminology stuff into the CQL engine (and not break the unit tests), we need to add support for this feature.

Specifically, from https://www.hl7.org/fhir/valueset.html#compositions, do this:

  • valueSet(s) only: Codes are 'selected' for inclusion if they are in all the referenced value sets
  • valueSet and System: Codes are 'selected' for inclusion if they are selected by the code system selection (after checking for concept and filter) and if they are in all the referenced value sets

We currently only accept a single referenced value set (while the first bullet requires value sets), and we don't use the system to limit the codes coming in from the valueset isn the second bullet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants