Skip to content

Conversation

@benjelloun
Copy link
Contributor

No description provided.

@benjelloun benjelloun requested a review from a team as a code owner December 19, 2025 16:57
@github-actions
Copy link

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@kulnor
Copy link
Member

kulnor commented Dec 19, 2025

@benjelloun To make it clear that both the short and long version of the "dataType" is supported at the Field level, I would add a couple of entries as shown below (min and max).

Could you add an inline comment to the JSON (or you need this to be valid JSON?). YAML could be a better way make this human readable.

You may also add a link and recommend the use of the DDI Controlled Vocabulary.

          "annotation": [
            {
              "@id": "person/age/min",
              "value": 16,
              "dataType": "ddi-stats:a1d0ec6"
            },
            {
              "@id": "person/age/max",
              "value": 75,
              "dataType": "ddi-stats:8321e79"
            },
            {
              "@id": "person/age/mean",
              "value": 42.51,
              "dataType": {
                "@type": "sc:DefinedTerm",
                "termCode": "ArithmeticMean",
                "name": "Arithmetic Mean",
                "@id": "ddi-stats:7975ed0",
                "inDefinedTermSet": "http://rdf-vocabulary.ddialliance.org/cv/SummaryStatisticType/2.1.2/"
              }
            }
           ] 

}
```

The total count of persons is a statistic on the RecordSet. It references Wikidata's [Cardinality](https://www.wikidata.org/wiki/Q4049983) term. The mean is a statistic on the `person/age` field. It reference the DDI-CDI `SummaryStatisticType` vocabulary.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

references?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants