Skip to content
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

padepy stops calculating descriptors from .smi files #49

Closed
totocotonio1 opened this issue Aug 2, 2023 · 4 comments
Closed

padepy stops calculating descriptors from .smi files #49

totocotonio1 opened this issue Aug 2, 2023 · 4 comments
Assignees

Comments

@totocotonio1
Copy link

Hi

padepy stops calculating descriptors from .smi files, the problem is not related to an error of a smile string, it stops calculating anywhere in the smi file, it should be fixed

Thank you

@tjkessler
Copy link
Member

Hi @totocotonio1,

Could you please provide a code snippet and a .smi file so I can help troubleshoot?

Best,
Travis

@tjkessler tjkessler self-assigned this Aug 2, 2023
@totocotonio1
Copy link
Author

totocotonio1 commented Aug 2, 2023

Hi @tjkessler

here is the script,

` pip install padelpy

from padelpy import padeldescriptor

padeldescriptor(mol_dir='/content/drive/MyDrive/test/test1.smi', d_file='/content/drive/MyDrive/test/descriptors1.csv', fingerprints=True)`

to be noted that I am using google colab

I am generating the files in myDrive

Thank you

test1.zip

@tjkessler
Copy link
Member

@totocotonio1,

I was able to generate the descriptors for your .smi file locally with everything located in the same directory:

from padelpy import padeldescriptor

padeldescriptor(
    mol_dir='test1.smi',
    d_file='descriptors1.csv',
    fingerprints=True
)

I've attached a ZIP file containing your .smi file, the resulting descriptors1.csv file, and the notebook I used: smiles_file_test.zip

I don't have too much experience with using PaDELPy in Google Colab, hopefully someone else can weigh in! Given that I was able to run everything locally, maybe the error is a file path issue...

Best,
Travis

@totocotonio1
Copy link
Author

Hi @tjkessler

Thank you very much for your help,

I will generate the descriptors locally

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants