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
Originally seen in #346.
Using a Bioportal ontology (like bioportal:SNOMEDCT) as an annotator works fine on its own.
If used in a value set expansion, e.g., with a reachable_from, it tries to download the corresponding semsql db.
This doesn't work if the semsql db does not exist.
It should throw a warning instead of trying to access a nonexistent resource.
INFO:ontogpt.engines.knowledge_engine:Grounding Elevated liver function tests. to SNOMEDCT:75540009; next step is to normalize
> /home/harry/ontogpt/src/ontogpt/engines/knowledge_engine.py(396)is_valid_identifier()
-> range_enum = sv.get_enum(e)
(Pdb) n
> /home/harry/ontogpt/src/ontogpt/engines/knowledge_engine.py(397)is_valid_identifier()
-> pvs = vse.expand_value_set(range_enum, sv.schema)
(Pdb) n
> /home/harry/ontogpt/src/ontogpt/engines/knowledge_engine.py(398)is_valid_identifier()
-> valid_ids = [pv.text for pv in pvs]
(Pdb) n
INFO:root:Locator: obo:SNOMEDCT
INFO:root:Ensuring gunzipped for https://s3.amazonaws.com/bbop-sqlite/SNOMEDCT.db.gz
INFO:pystow.utils:downloading with urllib from https://s3.amazonaws.com/bbop-sqlite/SNOMEDCT.db.gz to /home/harry/.data/oaklib/SNOMEDCT.db.gz
urllib.error.HTTPError: HTTP Error 404: Not Found
Originally seen in #346.
Using a Bioportal ontology (like bioportal:SNOMEDCT) as an annotator works fine on its own.
If used in a value set expansion, e.g., with a
reachable_from
, it tries to download the corresponding semsql db.This doesn't work if the semsql db does not exist.
It should throw a warning instead of trying to access a nonexistent resource.
This is the offending line:
ontogpt/src/ontogpt/engines/knowledge_engine.py
Line 396 in bba9692
Example as per #346, from debugger:
The issue here arises because oaklib's
ValueSetExpander
requires a specific config to work with Bioportal. Otherwise it assumes it's in OBO space.See https://github.com/INCATools/ontology-access-kit/blob/a51929160549b160831474472f93ebc9a5a22c01/src/oaklib/utilities/subsets/value_set_expander.py#L263-L295
The text was updated successfully, but these errors were encountered: