sdmx-twg / sdmx-ml

This repository is used for maintaining the SDMX-ML format specification
10 stars 4 forks source link

SDMX 3.0: implement "feature 013 Reorganising constraints" for SDMX-ML messages #16

Closed dosse closed 3 years ago

dosse commented 4 years ago

https://metadatatechnology.com/sdmx3/designs/013/approved/TF-2%20Reorganising%20Constraints%20v0.1.0.docx

dosse commented 3 years ago

In order to support the new requirement, it is proposed to:

structure message changes:

e.g. https://service-root/availability/ECB,EXR,1.0/A.EGP+RON..?startperiod=2010

<str:DataConstraint agencyID="ECB" id="9d6e73af-93e7-44e8-a6e9-b28380f05dc4" type="Actual">
      <str:ConstraintAttachment>
            <str:Dataflow>
            <Ref package="datastructure" agencyID="ECB" id="EXR" version="1.0" class="Dataflow"/>
            </str:Dataflow>
            <str:Filter>A.EGP+RON..?startperiod=2010</str:Filter>
      </str:ConstraintAttachment>
      <str:CubeRegion include="true">
            <KeyValue id="COU">
                  <Value>AUS</Value>
                  <Value>AUT</Value>
            </KeyValue>
            <KeyValue id="VAR">
                  <Value>INDEX_2000</Value>
            </KeyValue>
            <KeyValue id="TIME_PERIOD">
                  <TimeRange>
                        <StartPeriod isInclusive="true">1990-01-01T00:00:00</StartPeriod>
                        <EndPeriod isInclusive="true">2017-12-31T23:59:59</EndPeriod>
                  </TimeRange>
            </KeyValue>
      </str:CubeRegion>
</str:DataConstraint>

--> Implemented in structure schemas by J

data message changes: no impact

jgager commented 3 years ago

@dosse @stratosn Can you confirm that MetadataTargetRegion/KeyValue should be removed (e.g. MetadataTargetRegion should only contain Attribute to state the allowed values for MetadataAttributes)? Section 2.3.2 and 2.3.3 left this somewhat ambiguous.

stratosn commented 3 years ago

Hi @jgager , yes I confirm that only Attributes are included in the MetadataTargetRegion. At some point, there were some objections on the name of the latter (i.e., if it should stay 'MetadataTargetRegion'), but we haven't managed to come up with a better name.

jgager commented 3 years ago

@stratosn, thanks. I will leave the name as is for now and we can change if/when a better one is found.

dosse commented 3 years ago

Released