ValueSet.compose.include.valueSet definition "union" should be changed to "intersection"

XMLWordPrintableJSON

    • Type: Change Request
    • Resolution: Persuasive
    • Priority: Medium
    • FHIR Core (FHIR)
    • STU3
    • Terminology Infrastructure
    • ValueSet
    • Hide

      Update the definition of compose.include.valueset, [https://www.hl7.org/fhir/valueset-definitions.html#ValueSet.compose.include.valueSet

      FROM:

      "If multiple value sets are specified this includes the union of the contents of all of the referenced value sets."

      TO:

      "If multiple value sets are specified this includes the intersection of the contents of all of the referenced value sets."

      1. Add guidance for creating a union of value sets:
        1. Value Set Expansion http://hl7.org/fhir/valueset.html#expansion 
        2. Change the existing text in this section (#2) 
          1. FROM: For each valueSet, find the referenced value set by ValueSet.url, expand that (e.g., using the $expand operation: GET [base]/ValueSet/$expand$url=[compose.include.valueSet]), and add it to the result set."
          2. TO: For each valueSet, find the referenced value set by ValueSet.url, expand that (e.g., using the $expand operation: GET [base]/ValueSet/$expand$url=[compose.include.valueSet])to produce a collection of result sets.
            3. Add the intersection of the result set from the system *step 1) and all of the result sets from the value sets (step 2) to the expansion.
        3. change the last bullet text 
          1. FROM: "For each compose.exclude, follow the same process as for compose.include, but remove any codes from the result set, instead of adding them."
          2. TO: "For each compose.exclude, follow the same process as for compose.include, but remove codes from the expansion in step 3 instead of adding them."

       

      Voted as part of a block vote announced on Zulip and Vocab Listserve. Documented here:

      https://confluence.hl7.org/pages/viewpage.action?pageId=86969176

       

      Show
      Update the definition of compose.include.valueset, [https://www.hl7.org/fhir/valueset-definitions.html#ValueSet.compose.include.valueSet FROM: "If multiple value sets are specified this includes the union of the contents of all of the referenced value sets." TO: "If multiple value sets are specified this includes the intersection of the contents of all of the referenced value sets." Add guidance for creating a union of value sets: Value Set Expansion  http://hl7.org/fhir/valueset.html#expansion   Change the existing text in this section (#2)  FROM: For each valueSet, find the referenced value set by ValueSet.url, expand that (e.g., using the  $expand  operation: GET  [base] /ValueSet/$expand$url= [compose.include.valueSet] ), and add it to the result set." TO: For each valueSet, find the referenced value set by ValueSet.url, expand that (e.g., using the  $expand  operation: GET  [base] /ValueSet/$expand$url= [compose.include.valueSet] )to produce a collection of result sets. 3. Add the intersection of the result set from the system *step 1) and all of the result sets from the value sets (step 2) to the expansion. change the last bullet text  FROM: "For each  compose.exclude , follow the same process as for  compose.include , but remove any codes from the result set, instead of adding them." TO: "For each  compose.exclude , follow the same process as for  compose.include , but remove codes from the expansion in step 3 instead of adding them."   Voted as part of a block vote announced on Zulip and Vocab Listserve. Documented here: https://confluence.hl7.org/pages/viewpage.action?pageId=86969176  
    • Rob McClure/Carmela Couderc: 8-0-1
    • Clarification
    • Non-substantive
    • R5

      Per the definition of compose.include.valueset,, multiple valuesets listed in an include.valueset reference results in a UNION of members. However, composition rules here, indicate that within compose.include, multiple criterion result in the intersection of the criterion...which appears to be contradictory. Thus, the group (VSD Compliant Profile on Value Set team) is unaware of how existing elements can be used to result in the intersection of two references value sets.

      Update:10/27/19: Grahame has confirmed that the definition of compose.include.valueset appears to be erroneous, when it was moved under compose.include (previously it was under compose), _ the definition was supposed to be updated _ FROM:

      "If multiple value sets are specified this includes the union of the contents of all of the referenced value sets."

      TO

      "If multiple value sets are specified this includes the intersection of the contents of all of the referenced value sets."

      1. Update the element definition

      2. Add guidance for creating a union of value sets - modeling the VSAC approach of uisng multiple include statements vs: multiple include.valueSet instances.

      Detailed notes can be found on Row 78 of the VSDP Resource Element List.

            Assignee:
            Robert Hausam
            Reporter:
            Carmela Couderc
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: