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

About context #147

Open
cmaumet opened this issue Nov 22, 2024 · 3 comments
Open

About context #147

cmaumet opened this issue Nov 22, 2024 · 3 comments

Comments

@cmaumet
Copy link
Collaborator

cmaumet commented Nov 22, 2024

Update proposal for BIDS Prov (BEP028)

by @yarikoptic in #125 (comment)

@cmaumet
Copy link
Collaborator Author

cmaumet commented Nov 25, 2024

@yarikoptic Thanks for the suggestion. I do understand the value of simplifying the information that is required in BIDS-Prov and I am happy to discuss how we can achieve that (simplification is indeed a driving idea in the specification of BIDS-Prov).

it would be for the bids-prov tooling to extract those, equip with ids and add to the graph

Relying on BIDS-Prov tooling to recreate a valid JSON-LD from BIDS-Prov documents is to me a point of weakness. This would require sustained effort on the BIDS-Prov tooling and, unfortunately, I think that long-term support is not guaranteed (although I hope this will happen clearly).

I am keen for the BIDS-Prov document themselves to be valid JSON-LD as this will make it possible to benefit from the JSON-LD ecosystem of tools.

  • This probably means we have to keep the context in (unless we can have the context in a separate json and concatenate both files?).
  • About the identifiers, this is indeed something we could look into (using blank node) but this will be limited to the cases where we do not need to cross-reference a particular object (in practice I am not sure this will happen often).

@yarikoptic
Copy link
Contributor

I think that particular quote relates to

I agree that for a full formed .jsonld there should be @context defined within -- we should not lobotomize .jsonld if it is a .jsonld (not just .json).

I only wonder if there is a way to centralize @context reference somehow, e.g. at the top of the bids dataset and then have relative URLs pointing to it. If not -- I am ok. But might be worth bids-validator check to ensure that @context is consistent across all present .jsonld files and warn otherwise. Overall I think also relates to

@yarikoptic
Copy link
Contributor

if we are to avoid possible divergence from different .jsonld files pointing to different @context -- could be a job for BIDS validator (jsonld component) to warn if that happens. This way we would stick with duplicated explicit @context across all jsonld files.

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

No branches or pull requests

2 participants