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

updates to PandasTools.LoadSDF #599

Merged
merged 2 commits into from
Sep 4, 2015
Merged

updates to PandasTools.LoadSDF #599

merged 2 commits into from
Sep 4, 2015

Conversation

adalke
Copy link
Contributor

@adalke adalke commented Sep 4, 2015

The SDF loader is now about 2.4x the original performance.

If the filename ends with ".gz" (case-insenitive) then the file will be opened as a gzip file.

Support unicode filenames as well as byte filenames.

If a file object is passed in, then don't close it on exit - the caller is responsible for that.

If the molecule is empty, return an empty data frame instead of None.

The SDF loader is now about 2.4x the original performance.

If the filename ends with ".gz" (case-insenitive) then the file
will be opened as a gzip file.

If a file object is passed in, then don't close it on exit - the
caller is responsible for that.

If the molecule is empty, return an empty data frame instead of None.
@greglandrum greglandrum added this to the 2015_09_1 milestone Sep 4, 2015
greglandrum added a commit that referenced this pull request Sep 4, 2015
updates to PandasTools.LoadSDF
@greglandrum greglandrum merged commit 755a7a1 into rdkit:master Sep 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants