-
Notifications
You must be signed in to change notification settings - Fork 2
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
Problems using pheval #347
Comments
@seoanezonjic Thank you for your message, the whole team is OOO at the moment, but someone will come back to you when they are back! |
Thank you @matentzn and @yaseminbridges , I''ll wait to the team and in the meantime I'm playing with the platform. In fact, another problem that I faced is that I'm executing in my university cluster but the computing nodes has not access to internet and the script create_noisy_phenopackets.py fails in the following line: |
Hi, @seoanezonjic! Thank you for documenting these problems in detail! I think a lot of these issues may be due to the MakeFile pipeline needing to be updated to what we use in the paper. @souzadevinicius can you confirm? |
Hello, @seoanezonjic, Thank you for your message. I acknowledge that the PhEval pipeline documentation is outdated. I am currently working on updating it. In the meantime, you can utilize a pipeline described in this repository: https://github.com/monarch-initiative/monarch_pheval. The repository contains different types of experiments, which can be edited by modifying the PhEval Config YAML File. To execute all experiments outlined in the config file, please follow two steps provided in the pipeline documentation:
I apologize for any inconvenience caused by the outdated documentation. If you happen to have any other issues, please feel free to reach out to me. Thank you. |
@souzadevinicius is the pheval Makefile needed any more if the monarch-pheval repo exists? |
No, it's not, @julesjacobsen. I'm updating the PhEval pipeline documentation and removing these unnecessary files. |
Hi, @seoanezonjic! Just a brief update:
These changes are implemented in v0.4.0 |
Hi @souzadevinicius and @yaseminbridges
The line that makes reference:
If you could give some clue to fix this error I would thank you a lot. |
Apologies, just to clarify, did you face these errors when executing the pipeline here: https://github.com/monarch-initiative/monarch_pheval? For the time being, you should use the instructions outlined in that repo and tailor it to the experiments that you want to run and ignore the docs regarding the MakeFile pipeline in this repo. @souzadevinicius is OOO but he will be able to advise you further on your issues with the MakeFile once he returns |
Hi @yaseminbridges |
Hi @seoanezonjic
The MakeFile pipeline will automatically implement this command in its pipeline so you do not need to do so manually if you are planning to run the MakeFile. |
Hi @yaseminbridges
I hope that now it's clear my problem and I'm very sorry for the confusion |
Hi, @seoanezonjic. I hope you have successfully run the PhEval pipeline. In any case, the https://monarch-initiative.github.io/pheval/executing_a_benchmark/ has been updated. If you want to explore the pipeline process further, we have a dedicated repository available at https://github.com/monarch-initiative/monarch_pheval. Thank you! |
Hi @souzadevinicius |
Hello, Pedro. You're completely right about your suggestion of adding j2cli as a dependency and we did it. Additionally, in the updated documentation we don't mention j2 anymore because everything is encapsulated in Monarch PhEval repository. The Makefile generation is also fixed. |
Hi @souzadevinicius
This error I'm unable to resolve it. Any suggestion? |
Hello, @seoanezonjic. I recommend generating a new Makefile. You can do this by cloning the monarch_pheval repository and regenerating the Makefile. Here are the steps to follow:
Let me know if you have any questions! |
Hi @souzadevinicius
I've got this error:
A documentation explaining how works the file ./resources/pheval-config.yaml and how must be used would be very appreciated (not only examples but each section and attribute described to launch exomiser, lirical, gado and phen2gene programs I think that each one needs specific configuration). |
Hello, @seoanezonjic. We are almost there. Your Makefile is correct.
I'm adding to the documentation to make it clearer |
Hi @souzadevinicius
This I 've fixed editing runners/configurations/exomiser-14.0.0-2402_phenotype.config.yaml and change 2402 to "2402" but then the following error arises:
I've debugged line 117 from pheval_exomiser/run/run.py and a simple print as following: print(f"_{config.exomiser_software_directory}_", file=sys.stderr)
print(f"_{input_dir.joinpath(config.exomiser_software_directory)}_", file=sys.stderr) Shows this:
This is a path error because exomiser is installed in the runners folder as I can see in my local installation of the monarch-pheval. Maybe is a misconfiguration in my yaml file? |
Yes maybe it is, please share your config.yaml file. |
Hi @souzadevinicius directories:
data: data
tmp: data/tmp
corpora:
- id: lirical
custom_variants:
- id: no_phenotype
configs:
- tool: exomiser
id: exomiser-14.0.0-2402_phenotype
version: 14.0.0
runs:
- tool: exomiser
corpus: lirical
corpusvariant: default
version: 14.0.0
phenotype: "2402"
configuration: exomiser-14.0.0-2402_phenotype
|
The problem is in
You need to specify where the exomiser is located by this
I have the exomiser installation inside the config folder in my local example. My current local structure in the configurations folder: 📦configurations The Exomiser content is located in the configurations folder, so the |
Hi @souzadevinicius I suggest you to think and check that monarch_pheval repository will make a download, installation and execution steps in one Makefile at least with exomiser but with no user configuration (all automatized). Once the user is able to check that the system works in his/her environment, he/she could toy with the configuration to adapt to his/her needs. |
Hi Pheval authors
Recently I've seen your paper in Researchgate and I'm very interested so I gave a try. Following the instructions here https://monarch-initiative.github.io/pheval/exomiser_pipeline (I use a previous exomiser version installed in my cluster) I've have the following problems:
Once I've succesfully executed the pipeline, what is the next step to obtain the performance figures shown in the paper?
Thank you very much in advance
Pedro Seoane
The text was updated successfully, but these errors were encountered: