You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Based on KG-Microbe builds there are terms from OBO ontologies (eg ChEBI and GO) which get dropped from the transform step because KGX is rejecting terms with no 'name' attribute.
ChEBI Name N-acetyl-β-D-glucosaminyl-1,3-β-D-galactosyl-1,4-N-acetyl-D-glucosaminyl group
Describe the solution you'd like
Ideally OBO ontologies would have a name for each term, but it looks like this is not a requirement/not enforced at OBOFoundry (or there is a KGX obo json parsing issue?). In the interim, and to be more flexible for inputs, can KGX not require a 'name' attribute, ie maybe just a warning?
The text was updated successfully, but these errors were encountered:
And here is the line for the downstream transform code starting from 'obojson':
transform(inputs=[data_file], input_format='obojson', output= os.path.join(self.output_dir, name), output_format='tsv')
Is your feature request related to a problem? Please describe.
Based on KG-Microbe builds there are terms from OBO ontologies (eg ChEBI and GO) which get dropped from the transform step because KGX is rejecting terms with no 'name' attribute.
The message is:
"Skipping line as it does not have a name field: CHEBI 21588 biolink:ChemicalSubstance chebi.json http://purl.obolibrary.org/obo/CHEBI_21588"
Looking at the CHEBI page for this chemical, it does display a name so perhaps there is an issue with parsing obo json?
https://www.ebi.ac.uk/chebi/searchId.do?chebiId=CHEBI:21588
ChEBI Name N-acetyl-β-D-glucosaminyl-1,3-β-D-galactosyl-1,4-N-acetyl-D-glucosaminyl group
Describe the solution you'd like
Ideally OBO ontologies would have a name for each term, but it looks like this is not a requirement/not enforced at OBOFoundry (or there is a KGX obo json parsing issue?). In the interim, and to be more flexible for inputs, can KGX not require a 'name' attribute, ie maybe just a warning?
The text was updated successfully, but these errors were encountered: