Closed cmungall closed 1 month ago
Additional steps/options:
malco
)This will be run from https://github.com/monarch-initiative/malco
Also don't want to match to grouping classes like OMIM:MTHU068459
(Ataxia)
To improve grounding:
Easiest way to get phenotypic series from OMIM is to relax the ID filter.
I believe this should do everything malco
needs right now - if so, I will merge.
If we want to attempt gene inference then that can go in its own PR.
I'm seeing this error when running this on ontogpt 21813e4 on a small test set:
(.venv) ~/PythonProject/malco_new/prompts add_code_to_dl_phenopacket_store $ poetry update
Updating dependencies
Resolving dependencies... (29.9s)
Package operations: 0 installs, 1 update, 0 removals
• Updating ontogpt (0.3.11 5b4159e -> 0.3.11 21813e4)
Writing lock file
(.venv) ~/PythonProject/malco_new/prompts add_code_to_dl_phenopacket_store $ ontogpt run-multilingual-analysis --output=test.yaml /Users/jtr4v/PythonProject/malco_new/prompts/et/ /Users/jtr4v/PythonProject/malco_newoutputdir/
WARNING:ontogpt.clients:llm_gpt4all module not found. GPT4All support will be disabled.
WARNING:ontogpt.engines.knowledge_engine:GPT4All client not available. GPT4All support will be disabled.
Traceback (most recent call last):
File "/Users/jtr4v/PythonProject/malco_new/.venv/bin/ontogpt", line 8, in <module>
sys.exit(main())
^^^^^^
File "/Users/jtr4v/PythonProject/malco_new/.venv/lib/python3.11/site-packages/click/core.py", line 1157, in __call__
return self.main(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jtr4v/PythonProject/malco_new/.venv/lib/python3.11/site-packages/click/core.py", line 1078, in main
rv = self.invoke(ctx)
^^^^^^^^^^^^^^^^
File "/Users/jtr4v/PythonProject/malco_new/.venv/lib/python3.11/site-packages/click/core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jtr4v/PythonProject/malco_new/.venv/lib/python3.11/site-packages/click/core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jtr4v/PythonProject/malco_new/.venv/lib/python3.11/site-packages/click/core.py", line 783, in invoke
return __callback(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jtr4v/PythonProject/malco_new/.venv/lib/python3.11/site-packages/ontogpt/cli.py", line 1208, in run_multilingual_analysis
multilingual_analysis(input_data_dir=input_data_dir,
File "/Users/jtr4v/PythonProject/malco_new/.venv/lib/python3.11/site-packages/ontogpt/utils/multilingual.py", line 30, in multilingual_analysis
output = codecs.open(output, "wb", encoding="utf-8")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "<frozen codecs>", line 918, in open
TypeError: expected str, bytes or os.PathLike object, not LazyFile
aha, thought I had fixed that error, but evidently not. Fix incoming.
@justaddcoffee please give it another try
thanks @caufieldjh! works as advertised now
Excellent. Merging.
Some of the grounding is still fuzzier than it needs to be - or rather, it shouldn't include partial matches. Example:
OMIM:MTHU001678
shouldn't be in these because that's just "X-linked", or a partial match.