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

[WIP] Convert to LinkML #9

Merged
merged 14 commits into from
Aug 27, 2024
Merged

[WIP] Convert to LinkML #9

merged 14 commits into from
Aug 27, 2024

Conversation

sbliven
Copy link
Member

@sbliven sbliven commented Aug 21, 2024

Convert schemas to linkml (#5).

This is WIP to test the github actions.

Yves Ulrich Tittes and others added 14 commits July 10, 2024 16:23
- Temporarily remove tomoacquisition & shunt tomography slots into
  acquisition
- Remove aliases (error: alias not permitted in slot_usage)
- Distinguish temperature_range (class) from [Tt]emperature slots
- Replace `number` range with `float`
- rename problematic identifiers (name, class)
The following now pass (with warnings):

    make all
    make lint

Some tests still fail.

Summary of changes:

- Updated dependencies
- Use distinct `id` for each schema
- Remove `default_prefix: ex` that was causing collisions
- (Breaking) Move some utility types to `types` (Range, Series, etc).
  These should be replaced with versions from existing ontologies.
- (Breaking) Fix name collisions between schemas:
  - type -> instrument_type | molecular_type (with alias: type)
  - temperature -> temperature_range (alias: temperature)
- Generate the project (`make all`)
- Fix EMDataset-001.yaml so it validates
- (Breaking) Replace `type` with `instrument_type` or `molecular_type`.
  Otherwise it breaks python tests.
- (Breaking) Remove multivalued from institution
- Fix datetime range


Running `rm -r examples/output;  make test-examples` still fails
@sbliven sbliven mentioned this pull request Aug 26, 2024
@ytittes ytittes merged commit 0b2ab23 into main Aug 27, 2024
0 of 3 checks passed
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.

2 participants