-
Notifications
You must be signed in to change notification settings - Fork 6
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
Implement the semantic type expansion #13
Comments
Once implemented (@twktheainur looks into this), assign to @vemonet to add the following sentence: |
After checking with @mdorf from NCBO this is actually partially implemented with the &expand_semantic_types_hierarchy=true option. returns D005123 that has semantic type 023 |
However, there the current implementation is not transitive... i.e., all the childrena are not inclulded. Does not return anything where as it should...as the hierarchy is Will create an issue on NCBO tracker for this. |
See https://jira.bmir.stanford.edu/browse/NCBO-1838 for fix |
@mdorf Any news on this ? |
I haven't worked on this yet. The priority at this point is a) Re-designed BP Homepage, b) AllegroGraph, c) CEDAR Enhancements. NCBO-1838 is marked as "Low" priority. Let me know if this needs to be elevated (I'll need to coordinate with John). |
is it done @jonquet ? |
We can address this with @Bilelkihal |
When using the Annotator and filtering the result by smeantic types, the hiearchy of the semantic network must be used to include all the sub classes (transitive) of the selected semantic types.
For instance, if I select T057
http://bioportal.lirmm.fr/ontologies/STY/?p=classes&conceptid=http%3A%2F%2Fpurl.lirmm.fr%2Fontology%2FSTY%2FT058
then it is equivalent to select:T058, T060, T061, T059
This must be implemented in the Java proxy (i.e. this project) by automatically expanding the parameter
&semantic_types=T058
to
&semantic_types=T058,T060,T061,T059
The text was updated successfully, but these errors were encountered: