-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add full provenance info to metadata.json files #411
Comments
See Entity Issue: hubmapconsortium/entity-api#604 and PR: hubmapconsortium/entity-api#612 |
@ChuckKollar based on our conversation, here is what I think will be a simpler and more efficient solution:
|
PR: #481 |
Move writing of metadata.json to the end of dataset publish endpoint PR: #497 |
Jonathan has requested that the full provenance metadata information be included in the metadata.json files that are currently being generated (which only include the actual data metadata that we obtain during dataset ingest).
Both for the endpoint that will generate a metadata.json file for a dataset, and for the endpoint at publication time change the contents of the metadata.json file. Include the full json response of the entity-api
/entities/<dataset id>
endpoint and add to that lists of the full responses (from entity-api) for all associateddonors
,organs
andsamples
.See example cypher below to obtain the uuids of these associated entities, then make calls to the entity-api
/entities/<entity id>
endpoint to get the json for inclusion in the list of each type. These will be included as an array with one item per entity-api response with the keys:donor
,organ
,sample
.Also, of note-- the entity-api endpoint
/datasets/<dataset id>/organs
could be used to get the organ information, but similar endpoints don't exist for Sample and Donor... A new endpoint will be created in entity-api to retrieve the 'organ', 'donor', and 'sample' uuids (see: hubmapconsortium/entity-api#604).So, adding arrays to the standard entity-api endpoint json response for
/entities/<dataset id>
for each ofdonors
,organs
andsamples
is what we need written to the metadata.json file now, instead of just the dataset.ingest_metadata.metadata, as it is now.Attached example metadata.json for fa238ec2a83302fb4af92442c3683a23, with added donors, samples and organs
fa238ec2a83302fb4af92442c3683a23-sample-metadata.json
Remember that this change must be effected in two endpoints in
ingest-api
:Remember to make the necessary changes to the
OpenAPI.yaml
foringest-api
.The text was updated successfully, but these errors were encountered: