Closed david4096 closed 7 years ago
Copying response from #710:
This makes sense: to have a tag-bag field that strictly represents Ontology Terms, although the semantic context of those terms can be lost if no name is provided.
Please also consider the approach of allowing tagging via Ontology Term (and others) in a generic attributes field by upgrading the info field. This would allow a data curator to define named tag bags on a biosample with some more context without adding new named fields to the message. This replaces the info field and both approaches are not exclusive. PR for this feature here.
I'm +1 for this approach as it solves the immediate issue of specifying tissue type for TCGA data.
+1 for the merge
+1 Looks good to me.
Seconding my +1. When loading the TCGA biosamples it appears some are labeled with more than one disease, making the singly valued disease
field not especially helpful!
So merge depending on the integration team. @david4096 @kozbo?! (If no objections to the attribute's name).
Hi @david4096,
We talked further with @mbaudis and would like to propose replacing the _samplecharacteristics attribute with the characteristics object, with structure:
characteristics: [
{
description: “squamous cell carcinoma, base of tongue, stage 2”,
type: phenotype (could be organism, disease...)
repeated OntologyTerm ontologyTerms: [
{
ontologyId: “http://purl.obolibrary.org/obo/DOID_0050865”,
term: “tongue squamous cell carcinoma”,
},
{
ontologyId: “http://purl.obolibrary.org/obo/UBERON_0006919”,
term: “tongue squamous epithelium”,
},
{
ontologyId: “http://purl.obolibrary.org/obo/UBERON_0010033”
term: “posterior part of tongue”,
},
],
}
]
where each ontologyTerm is simplified in the above, but would in fact be the OntologyTerm structure as we agreed on at https://github.com/ga4gh/schemas/pull/694/files (and include version and source).
Neat! That's certainly a more flexible way of describing characteristics! Could you take a look at https://github.com/ga4gh/schemas/pull/700 ? I believe it is attempting to perform a similar facility and would apply across the API. The additions you've made from what I can tell are restricting the characteristics to ontology terms, and providing a description and controlled vocabulary for the type
.
Following the discussions at Vancouver: Closing this in favour https://github.com/ga4gh/schemas/pull/725.
Reopened from #710 to remove extraneous commits. @mbaudis if you're happy with it you can close the other.