Skip to content

Schemas for a more fine-grained representation of services#62

Merged
lzehl merged 5 commits intov3from
service-interfaces
Feb 23, 2026
Merged

Schemas for a more fine-grained representation of services#62
lzehl merged 5 commits intov3from
service-interfaces

Conversation

@apdavison apdavison changed the base branch from v2 to v3 November 7, 2025 17:16
@apdavison apdavison marked this pull request as ready for review January 28, 2026 16:13
"accessibility": {
"_instruction": "Add the accessibility of this deployed interface.",
"_linkedTypes": [
"controlledTerms.ProductAccessibility"
Copy link
Member

Choose a reason for hiding this comment

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

only Accessibility now (since the schema can be also used for non-products)

Copy link
Member

Choose a reason for hiding this comment

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

maybe we don't want to call this service deployment but rather deployment only? and also making the service optional? or did we say now everything with an interface is a service?

Copy link
Member

Choose a reason for hiding this comment

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

hasPart for implemented configurations need to be covered in here: CommonCoordinateFrameworkVersion, AnatomicalAtlasVersion, ModelVersion (maybe include also LaunchConfiguration?)

Copy link
Member

Choose a reason for hiding this comment

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

now solved through dependsOn and uses

"type": "array",
"minItems": 1,
"uniqueItems": true,
"_embeddedTypes": [
Copy link
Member

Choose a reason for hiding this comment

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

linkedType and besides DeployedInterface should also link to all researchProductVersions; either needs a distinct list of types or a new categegory for all RPVs and DeployedInterface

Copy link
Member

@lzehl lzehl left a comment

Choose a reason for hiding this comment

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

@olinux @apdavison please check the updated uses and dependsOn and see if you agree

@lzehl lzehl merged commit ff7ef18 into v3 Feb 23, 2026
2 checks passed
@lzehl lzehl deleted the service-interfaces branch February 23, 2026 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants