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

Compose "full" example for components schema #44

Merged
merged 5 commits into from
Feb 21, 2024
Merged

Compose "full" example for components schema #44

merged 5 commits into from
Feb 21, 2024

Conversation

mih
Copy link
Contributor

@mih mih commented Feb 18, 2024

See src/examples/datalad-dataset-components/ContainerSE-DatasetWFiles.yaml

It works in principle, but I was not yet able to get the description of an AnnexedFileSE's distribution to work (see comments inside).

This is strange, because this part is more or less taken verbatim from the datalad-dataset-version schema, and
src/examples/datalad-dataset-version/DataladDatasetVersionSE-full.yaml shows it to be working properly.

As elaborated on inside, the JSON schema code generator does not include a class that is required for validation. However, it does reference it.

Update: I possible fix or workaround was found and is subject to review at linkml/linkml#1935

Update: I include linkml-patching now, and enabled the full example.

See src/examples/datalad-dataset-components/ContainerSE-DatasetWFiles.yaml

It works in principle, but I was not yet able to get the description of
an AnnexedFileSE's `distribution` to work (see comments inside).

This is strange, because this part is more or less taken verbatim from
the `datalad-dataset-version` schema, and
`src/examples/datalad-dataset-version/DataladDatasetVersionSE-full.yaml`
shows it to be working properly.

As elaborated on inside, the JSON schema code generator does not include
a class that is required for validation. However, it does reference it.
Include a patch that fixes a JSON schema generation problem that blocks
progress for us.
Becomes possible after applying the linkml patch introduced in the
previous commit.
@mih mih merged commit f951b68 into main Feb 21, 2024
3 checks passed
@mih mih deleted the components-demo branch February 21, 2024 08:29
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.

1 participant