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

Update sascorer.py #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Mahmoud-A1i
Copy link

Fixed an issue with the imports.

Changed 'from rdkit.six import iteritems'
To 'from six import iteritems'

Fixed an issue with the imports.

Changed 'from rdkit.six import iteritems'
To 'from six import iteritems'
@supershiye
Copy link

Does it work on your side? I changed the 'append' and 'six' issues. And it seems the PD readcsv still not working.

@Mahmoud-A1i
Copy link
Author

Does it work on your side? I changed the 'append' and 'six' issues. And it seems the PD readcsv still not working.

Worked with me. Can I see your error message?

@supershiye
Copy link

Does it work on your side? I changed the 'append' and 'six' issues. And it seems the PD readcsv still not working.

Worked with me. Can I see your error message?

Thank you! I figured it out. I did not install git lfs. So the dataset files were not downloaded.

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

Successfully merging this pull request may close these issues.

2 participants