Skip to content

spec: 1.2+202609 version - #163

Merged
NicolasCARPi merged 5 commits into
masterfrom
nico/spec
Sep 23, 2026
Merged

NicolasCARPi merged 5 commits into
masterfrom
nico/spec

Conversation

@NicolasCARPi

Copy link
Copy Markdown
Contributor

No description provided.

* master:
  readme: add ELNPack and ELN metadata viewer (#162)
  Align validation flow: unified runChecks, logo, and tests
  Add a web .eln checker that reuses the test suite checks
  Validate .eln single root folder (#156)
  Clarify child Dataset relationships (#155)
  Validate array-valued node types (#154)
  spec: clarify type and genre fields (#153)
  Add BenchLineage ELN example (#152)
  chore: update SciLog as supporting import (#151)

@FlorianRhiem FlorianRhiem left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me!

@SteffenBrinckmann

SteffenBrinckmann commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator
  1. Clean up date-time format
72c72
<   "dateCreated": "2022-05-30T12:25:36+0200",
---
>   "dateCreated": "2022-05-30T12:25:36+02:00",
175,176c183,184
<   "dateCreated": "2022-05-29 16:17:38",
<   "dateModified": "2022-05-29 16:17:57",
---
>   "dateCreated": "2022-05-29T16:17:38",
>   "dateModified": "2022-05-29T16:17:57",
  1. Update version 1.1 -> 1.2 in line 81

  2. Clarify the hasPart rule: split across lines

112,113d111
< Subsequently, all the remaining nodes are assigned a `@type` of either `Dataset` for directories or `File` for individual files. And the `@id` corresponds to something in the `hasPart` of `./`.
< 
117c115
< If a Dataset references child Datasets (_e.g._ a parent experiment with child experiments), it SHOULD list them in its `hasPart` section. Each child Dataset that is meant to be imported MUST also be listed directly in the `hasPart` of `./`; a reference from its parent alone does not mark it for import.
---
> Every Dataset or File entity intended for import MUST be referenced directly in `./`’s hasPart. An entity that belongs to a Dataset MUST also be referenced in that Dataset’s hasPart, to record the hierarchy. An entity not intended for import, such as a previous version of a file, MAY be referenced only in its containing Dataset’s hasPart and MUST NOT be referenced directly by the Root Dataset.
  1. We had "Comments exist .. at the root node". That is wrong "Comments exist ... in graph"
150c148
< All nodes with a `@type` such as `Comment` or `Person` exist at the root node (once), and can be referenced via their `@id` in other parts.
---
> Contextual entities, such as Comment and Person, MUST be represented once as separate node objects in the top-level @graph. Dataset and File entities MAY reference them using their @id.
  1. Replace identifier with propertyID as it is given in schema.org. (Identifier has different meaning) Also add an example
163c161,171
<   * The `identifier` for a `@type: PropertyValue` node can be set to an IRI (e.g. the URL for an ontology entry, such as http://purl.org/dc/terms/instructionalMethod) for specifying the meaning of this node.
---
>   * A PropertyValue node SHOULD use propertyID to identify the represented property. propertyID MAY be an ontology IRI. The recorded value SHOULD be given in value.
>     ``` json
>       {
>         "@id": "#air-temperature",
>         "@type": "PropertyValue",
>         "propertyID": "https://qudt.org/vocab/quantitykind/Temperature",
>         "name": "Air temperature",
>         "value": 21.5,
>         "unit": "C"
>       }
>     ```

@SteffenBrinckmann

Copy link
Copy Markdown
Collaborator

More items from our discussion today:

  1. Append to line 113 "All nodes with @type: Dataset SHOULD...."
    keywords are a comma separated list of words.

  2. Append to line 154 "additionalType:...."
    additionalType: {"@id":url} OR [{"@id":url}]

@NicolasCARPi

Copy link
Copy Markdown
Contributor Author

Thanks Steffen, I addressed your comments.

@SteffenBrinckmann SteffenBrinckmann left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Great

@NicolasCARPi
NicolasCARPi merged commit 2379f51 into master Sep 23, 2026
2 checks passed
@NicolasCARPi
NicolasCARPi deleted the nico/spec branch September 23, 2026 08:48
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