-
Notifications
You must be signed in to change notification settings - Fork 68
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
No such file or directory: 'data/LIGAND_RAW.tsv' #23
Comments
Hi, unfortunately the file is missing from the repository it seems. However, note that there is now an improved and self-contained version of DrugEx available here: https://github.com/CDDLeiden/DrugEx If you are still looking for this file specifically, I can also provide it. @XuhanLiu Do you think you could link to the new repository also in the README.md? Perhaps something along the lines: "This project was now upgraded to an installable package with tutorials and test data. You can find it here: https://github.com/CDDLeiden/DrugEx" Many thanks! |
Can you provide it to me. I am trying to re do it fom scratch and provide bew algorithms may be. I am preparing a thesis. Thank you |
@amiloudij We have versions of some old files here. The names are a bit mangled, but you should be able to find the file you are looking for in the |
'data/ligand_mf_brics.txt ERROR....no such file or directory |
'data/ligand_mf_brics.txt ERROR....no such file or directory |
This file should get generated if the processing of It is possible there is an error in the names of the files here: https://github.com/XuhanLiu/DrugEx/blob/master/train_graph.py So you might need to edit them. |
I dont have the file voc_ |
This one should be fine to use as well: https://github.com/CDDLeiden/DrugEx/blob/master/testing/clitest/data/voc_graph.txt |
By the way, if you are planning to add new algorithms and do not need to reproduce the experiments from the original paper exactly, I would really recommend the new packaged version again. It is more user friendly and also extensible. If you are willing to contribute your implementation to open source, we would love to help you get started with it as well. |
I see that you are using a file named voc_atom.txt in the code and we dont have it in the repo. |
You can see the one I mention here:
Just rename it. |
@martin-sicho Yes I want to contribute with you. What kind of help can you help me? |
I you want to use the updated repo, you can get all basic information with the tutorial: https://github.com/CDDLeiden/DrugEx/tree/master/tutorial We are still improving and updating it, but if you want to use the graph-based transformer or the RNN, you should be covered for most applications. The Python API already offers some customization, but if you want to modify the models a very specific way or add your own, let us know on the issue tracker and we can suggest how to approach it. For more info there is a README and documentation. |
how to handle that?
The text was updated successfully, but these errors were encountered: