-
Notifications
You must be signed in to change notification settings - Fork 41
The PSI-Mass Spectrometry ontology hierarchy does not look right in OLS website #623
Comments
I think there are multiple issues here:
In both cases 1 and 2 I strongly suggest the creators of the ontology to get in touch with @dosumis to discuss ways to address this.
|
OLS has been updated to fix (3) above. (1) and (2) still needs to be addressed by the designers of the PSIMass Spectrometry ontology. |
No, the issues are not solved still. Work in progress. |
Though we can update MS on OLS, it will still be broken on OLS while these are still an issue. |
Okay, thank you, @dosumis how can we address these issues? |
HI @edeutsch - Maybe a quick call to talk through the issues would be the most efficient way to proceed? If you're interested, we could help you switch to a build pipeline with QC to prevent issues like this from occurring. |
I don't see an OLS issue. OLS uses the MS PURL http://purl.obolibrary.org/obo/ms.owl to download the ontology. This ontology imports the entirety the PATO and STATO. This is why your top level looks so "untidy": There are many unsats as @henrietteharmse says: Open MS in Protege and debug it. Or use ROBOT explain:
This will help you identify the logical errors! Hope that helps! |
To improve the view in OLS, you can add preferred roots to your OLS config. This will set the terms that are visible at the top level by default. @serjoshua should be able to point you to documentation on how to do this. |
@dosumis Preferred roots will not sort out issues like the |
|
@matentzn Agreed about |
Dear colleagues,
https://www.ebi.ac.uk/ols/ontologies/ms
It seems that something is not right because in the first hierarchical level of the ontology, many terms from the ontology (which should not) appear and also the complete UNIT ontology as well. This is not right. I don't know whether it is the fault of the parser or we have formatted wrongly the OWL file in any of the recent changes.
Could you please help?
Thanks,
Juan
The text was updated successfully, but these errors were encountered: